-
Notifications
You must be signed in to change notification settings - Fork 424
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CARGO_BAZEL_REPIN does not work with lints.workspace = true
#2536
Comments
We generate a synthetic workspace Cargo.toml for the sake of dependency resolution, and it doesn't have a We can make it have one if needed, but I ask https://rust-lang.zulipchat.com/#narrow/stream/246057-t-cargo/topic/lints.2Eworkspace.20requires.20workspace.20to.20define.20lints to get some clarity on whether Cargo should actually be erroring in this case. |
I was running into the same issue, and I'm pretty sure it's because the current version of cargo-toml doesn't parse the workspace |
Addressed in #2551 |
Hi 👋
I use the https://blog.rust-lang.org/2023/11/16/Rust-1.74.0.html#lint-configuration-through-cargo feature, but it seems it does not work well with
cargo-bazel
. Here is the error.Do you have an idea ?
The text was updated successfully, but these errors were encountered: