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

uninlined_format_args: Ignore assert! and debug_assert! before 2021 edition #10055

Merged
merged 1 commit into from
Dec 10, 2022

Conversation

taiki-e
Copy link
Member

@taiki-e taiki-e commented Dec 10, 2022

Similar to #9605, but for assert! and debug_assert! macros. (non_fmt_panics lint triggers them)

changelog: [uninlined_format_args]: Do not inline assert! and debug_assert! macros before 2021 edition

r? @llogiq

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Dec 10, 2022
@llogiq
Copy link
Contributor

llogiq commented Dec 10, 2022

Thank you!

@bors r+

@bors
Copy link
Collaborator

bors commented Dec 10, 2022

📌 Commit e5010c9 has been approved by llogiq

It is now in the queue for this repository.

@bors
Copy link
Collaborator

bors commented Dec 10, 2022

⌛ Testing commit e5010c9 with merge c0130e4...

@bors
Copy link
Collaborator

bors commented Dec 10, 2022

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

@bors bors merged commit c0130e4 into rust-lang:master Dec 10, 2022
@taiki-e taiki-e deleted the uninlined_format_args branch December 10, 2022 10:05
oisyn added a commit to EmbarkStudios/rust-gpu that referenced this pull request Jan 5, 2023
This nightly includes a clippy fix that we need (rust-lang/rust-clippy#10055)
eddyb pushed a commit to EmbarkStudios/rust-gpu that referenced this pull request Jan 5, 2023
This nightly includes a clippy fix that we need (rust-lang/rust-clippy#10055)
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.

None yet

4 participants