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
Add new mixed_attributes_style
lint
#12354
Conversation
a1319bf
to
6edc3f9
Compare
Note for future self: run |
6edc3f9
to
1631ee3
Compare
And also dogfood... |
Ci passed. \o/ |
} | ||
|
||
#[allow(unused)] | ||
#[allow(unused)] |
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.
Attribute duplication might be worth another lint.
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.
Dully noted.
Thank you! @bors r+ |
Add new `mixed_attributes_style` lint Add a new lint to detect cases where both inner and outer attributes are used on a same item. r? `@llogiq` changelog: Add new `mixed_attributes_style` lint
💔 Test failed - checks-action_test |
Seems completely unrelated. Retry? :3 |
@bors retry |
Add new `mixed_attributes_style` lint Add a new lint to detect cases where both inner and outer attributes are used on a same item. r? `@llogiq` ---- changelog: Add new [`mixed_attributes_style`] lint
💔 Test failed - checks-action_test |
1631ee3
to
4a228ba
Compare
Rebased and allowed this lint in |
4a228ba
to
f301386
Compare
Rebased again. |
@bors r+ |
☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test |
Add a new lint to detect cases where both inner and outer attributes are used on a same item.
r? @llogiq
changelog: Add new [
mixed_attributes_style
] lint