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

respect #[allow] attributes in single_call_fn lint #12183

Merged
merged 1 commit into from Jan 22, 2024

Conversation

y21
Copy link
Member

@y21 y21 commented Jan 21, 2024

Fixes #12182

If we delay linting to check_crate_post, we need to use span_lint_hir_and_then, since otherwise it would only respect those lint level attributes at the crate root.
... maybe we can have an internal lint for this somehow?

changelog: respect #[allow] attributes in single_call_fn lint

@rustbot
Copy link
Collaborator

rustbot commented Jan 21, 2024

r? @dswij

(rustbot has picked a reviewer for you, use r? to override)

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

@dswij dswij left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM :)

@dswij
Copy link
Member

dswij commented Jan 22, 2024

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 22, 2024

📌 Commit ad4d90b has been approved by dswij

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Jan 22, 2024

⌛ Testing commit ad4d90b with merge 0b6e7e2...

@bors
Copy link
Collaborator

bors commented Jan 22, 2024

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing 0b6e7e2 to master...

@bors bors merged commit 0b6e7e2 into rust-lang:master Jan 22, 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.

#[allow(clippy::single_call_fn)] won't work
4 participants