-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Extend author lint #7894
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
Extend author lint #7894
Conversation
r? @camsteffen (rust-highfive has picked a reviewer for you, use r? to override) |
f73a596
to
2b87f8f
Compare
7a7fad1
to
7e9f966
Compare
73509df
to
b9437e1
Compare
b9437e1
to
6430fe4
Compare
☔ The latest upstream changes (presumably #7906) 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.
LGTM after rebase
6430fe4
to
c96cd35
Compare
Thanks! @bors r+ |
📌 Commit c96cd35 has been approved by |
Extend author lint changelog: * Print float and int suffixes * Print labels * Struct field checks * Repeat length expression check * Destructure method calls * Destructure closures
💔 Test failed - checks-action_test |
@bors retry (fixed changelog) |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
changelog: none