-
-
Notifications
You must be signed in to change notification settings - Fork 938
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
Fix *-no-redundant-*
false negatives for inset
shorthand
#6699
Conversation
🦋 Changeset detectedLatest commit: 95e24f0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
declaration-block-no-redundant-longhand-properties
false negatives for inset
shorthandinset
property shorthand
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.
@rayrw Thanks for the pull request. Almost LGTM, but I've left several comments about the code style. Could you please consider addressing them?
lib/rules/declaration-block-no-redundant-longhand-properties/__tests__/index.js
Outdated
Show resolved
Hide resolved
lib/rules/declaration-block-no-redundant-longhand-properties/__tests__/index.js
Outdated
Show resolved
Hide resolved
@ybiquitous Thanks for the reviews! |
inset
property shorthanddeclaration-block-no-redundant-longhand-properties
and shorthand-property-no-redundant-values
false negatives for inset
shorthand
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.
Thank you for the quick fix! LGTM 👍🏼
declaration-block-no-redundant-longhand-properties
and shorthand-property-no-redundant-values
false negatives for inset
shorthand*-no-redundant-*
false negatives for inset
shorthand
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, thank you.
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.
@jeddy3 Thanks for making the title and changelog much simpler! 👍🏼
Closes #6613
This PR updates these 2 rules to support the
inset
propertydeclaration-block-no-redundant-longhand-properties
shorthand-property-no-redundant-values