Skip to content

fix(lints): Improve ill_formed_attribute_input with better help message#154722

Open
JayanAXHF wants to merge 1 commit intorust-lang:mainfrom
JayanAXHF:fix/update-ignore-error-message
Open

fix(lints): Improve ill_formed_attribute_input with better help message#154722
JayanAXHF wants to merge 1 commit intorust-lang:mainfrom
JayanAXHF:fix/update-ignore-error-message

Conversation

@JayanAXHF
Copy link
Copy Markdown
Member

@JayanAXHF JayanAXHF commented Apr 2, 2026

Add a better help message to ill_formed_attribute_input for the cases where it matches the usage of #![allow].

For further details see #154667

closes #154667

cc: @Zalathar

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 2, 2026

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann, @JonathanBrouwer

@rustbot rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) 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 Apr 2, 2026
@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Apr 2, 2026

r? @petrochenkov

rustbot has assigned @petrochenkov.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 69 candidates
  • Random selection from 12 candidates

@jdonszelmann
Copy link
Copy Markdown
Contributor

r? me

@rustbot rustbot assigned jdonszelmann and unassigned petrochenkov Apr 2, 2026
@jdonszelmann
Copy link
Copy Markdown
Contributor

This change seems to have no tests. Did no tests change or do we not have a test for this new behavior yet?

@jdonszelmann
Copy link
Copy Markdown
Contributor

@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 2, 2026
…sage for the cases where it matches the usage of #![allow]
@JayanAXHF JayanAXHF force-pushed the fix/update-ignore-error-message branch from d60f71d to 94b8874 Compare April 2, 2026 18:10
@JayanAXHF
Copy link
Copy Markdown
Member Author

Oops forgot about that. @jdonszelmann I've added a test for this, lmk if there are any more changes to be made

@rustbot ready

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Apr 2, 2026
Copy link
Copy Markdown
Contributor

@jdonszelmann jdonszelmann left a comment

Choose a reason for hiding this comment

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

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors bot commented Apr 2, 2026

📌 Commit 94b8874 has been approved by jdonszelmann

It is now in the queue for this repository.

@rust-bors rust-bors bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Apr 2, 2026
rust-bors bot pushed a commit that referenced this pull request Apr 2, 2026
…uwer

Rollup of 4 pull requests

Successful merges:

 - #154444 (rustdoc ICE fix: When collecting `Deref` impls with their targets, skip the negative ones)
 - #154590 (Make #[cfg] suggest any or all on #[cfg(a, b)])
 - #154691 (core: Update the feature gate on `TryFrom<integer> for bool`)
 - #154697 (rustdoc: fix href of extern crates in search results)

Failed merges:

 - #154722 (fix(lints): Improve `ill_formed_attribute_input` with better help message)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-attributes Area: Attributes (`#[…]`, `#![…]`) S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

#[ignore(lint_name)] errors should suggest using #[allow(lint_name)] instead

4 participants