Skip to content

Merge rustc_lint/lints.rs into diagnostics.rs - #160755

Open
JonathanBrouwer wants to merge 1 commit into
rust-lang:mainfrom
JonathanBrouwer:rustc_lint_diags
Open

Merge rustc_lint/lints.rs into diagnostics.rs#160755
JonathanBrouwer wants to merge 1 commit into
rust-lang:mainfrom
JonathanBrouwer:rustc_lint_diags

Conversation

@JonathanBrouwer

Copy link
Copy Markdown
Contributor

Renaming diagnostics files to all be diagnostics.rs, per the decision in rust-lang/compiler-team#1003

r? @GuillaumeGomez

  • I did not use an LLM to create a change in this PR.
  • I used an LLM to create a change in this PR, and I have explained below how it was used.

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 8, 2026
@@ -108,7 +126,7 @@ pub(crate) struct BuiltinEllipsisInclusiveRangePatterns {
#[derive(Subdiagnostic)]
#[note("requested on the command line with `{$level} {$lint_name}`")]
pub(crate) struct RequestedLevel<'a> {
pub level: Level,
pub level: rustc_session::lint::Level,

@JonathanBrouwer JonathanBrouwer Aug 8, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was needed as there is now another type called Level in scope which was used in more places

View changes since the review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants