-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Write literal suggestion #6821
Write literal suggestion #6821
Conversation
r? @flip1995 (rust-highfive has picked a reviewer for you, use r? to override) |
7537128
to
a15f9ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for taking so long! This looks overall really good! I just have two NITs.
☔ The latest upstream changes (presumably #6914) made this pull request unmergeable. Please resolve the merge conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Needs rebase, otherwise LGTM.
Don't lint on a mixture of raw and regular strings Fix spans in format strings
b130ce0
to
6946cc1
Compare
6946cc1
to
d45873b
Compare
@bors r+ Thanks! |
📌 Commit d45873b has been approved by |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
fixes: #6768
changelog: Add suggestion to
write_literal
andprint_literal
lintschangelog: Change
use_debug
to point only at the format string