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

Don't allow derive macros to silence disallowed_macros #12267

Merged
merged 1 commit into from Feb 12, 2024

Conversation

Jarcho
Copy link
Contributor

@Jarcho Jarcho commented Feb 10, 2024

fixes #12254

The implementation is a bit of a hack, but "works". A derive expanding to another derive won't work properly, but we shouldn't be linting those anyways.

changelog: disallowed_macros: Don't allow derive macros to silence their own expansion

@rustbot
Copy link
Collaborator

rustbot commented Feb 10, 2024

r? @Alexendoo

rustbot has assigned @Alexendoo.
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

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Feb 10, 2024
@Alexendoo
Copy link
Member

Nice, would let people apply #[allow] to the item with the derive on it also

@bors r+

@bors
Copy link
Collaborator

bors commented Feb 11, 2024

📌 Commit ac7c6e5 has been approved by Alexendoo

It is now in the queue for this repository.

bors added a commit that referenced this pull request Feb 11, 2024
Don't allow derive macros to silence `disallowed_macros`

fixes #12254

The implementation is a bit of a hack, but "works". A derive expanding to another derive won't work properly, but we shouldn't be linting those anyways.

changelog: `disallowed_macros`: Don't allow derive macros to silence their own expansion
@bors
Copy link
Collaborator

bors commented Feb 11, 2024

⌛ Testing commit ac7c6e5 with merge 446974a...

@bors
Copy link
Collaborator

bors commented Feb 11, 2024

💔 Test failed - checks-action_test

@Alexendoo
Copy link
Member

@bors retry

@bors
Copy link
Collaborator

bors commented Feb 12, 2024

⌛ Testing commit ac7c6e5 with merge b0e7640...

@bors
Copy link
Collaborator

bors commented Feb 12, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: Alexendoo
Pushing b0e7640 to master...

@bors bors merged commit b0e7640 into rust-lang:master Feb 12, 2024
5 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

disallowed_macros false negatives for specific derive macros
4 participants