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

Make useless_format recognize format!("") #7801

Merged
merged 1 commit into from
Oct 21, 2021

Conversation

aDotInTheVoid
Copy link
Member

Closes #7796

changelog: [useless_format] Fix for false negitive for format!("")

@rust-highfive
Copy link

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @camsteffen (or someone else) soon.

Please see the contribution instructions for more information.

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Oct 11, 2021
@bors
Copy link
Contributor

bors commented Oct 21, 2021

☔ The latest upstream changes (presumably #7853) made this pull request unmergeable. Please resolve the merge conflicts.

@camsteffen
Copy link
Contributor

This is good after a rebase and remove the "add a semicolon" part - this is no longer needed after recent changes to rustc.

@aDotInTheVoid aDotInTheVoid force-pushed the empty-format branch 2 times, most recently from 720c8a6 to b799d4a Compare October 21, 2021 17:21
@camsteffen
Copy link
Contributor

Looks like we need #![allow(unused_must_use)] in the test.

@camsteffen
Copy link
Contributor

@bors r+

@bors
Copy link
Contributor

bors commented Oct 21, 2021

📌 Commit 081d0f8 has been approved by camsteffen

@bors
Copy link
Contributor

bors commented Oct 21, 2021

⌛ Testing commit 081d0f8 with merge 2ba1a6a...

@bors
Copy link
Contributor

bors commented Oct 21, 2021

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: camsteffen
Pushing 2ba1a6a to master...

@bors bors merged commit 2ba1a6a into rust-lang:master Oct 21, 2021
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.

clippy::useless_format ignores format!("")
4 participants