Skip to content
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

Deny warnings in CI #296

Merged
merged 1 commit into from
May 6, 2024
Merged

Deny warnings in CI #296

merged 1 commit into from
May 6, 2024

Conversation

tgross35
Copy link
Contributor

@tgross35 tgross35 commented May 6, 2024

The main crate already has #![deny(warnings)]. Set RUSTFLAGS in CI to enforce this for other crates in the workspace.

@tgross35
Copy link
Contributor Author

tgross35 commented May 6, 2024

Related to attempting to remove warnings in compiler-builtins rust-lang/compiler-builtins#603

@tgross35 tgross35 force-pushed the ci-deny-warnings branch 2 times, most recently from b9f81ee to cf82b33 Compare May 6, 2024 09:23
@tgross35 tgross35 force-pushed the ci-deny-warnings branch 4 times, most recently from 94a7f23 to c63717a Compare May 6, 2024 09:50
The main crate already has `#![deny(warnings)]`. Set RUSTFLAGS in CI to
enforce this for other crates in the workspace.
@tgross35 tgross35 force-pushed the ci-deny-warnings branch from c63717a to 32e7d04 Compare May 6, 2024 09:51
@Amanieu Amanieu merged commit 279e5f6 into rust-lang:master May 6, 2024
12 checks passed
@tgross35 tgross35 deleted the ci-deny-warnings branch May 6, 2024 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants