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

Fix false positives for valueless attribute selectors in selector-attribute-name-disallowed-list #5060

Merged
merged 1 commit into from Dec 8, 2020
Merged

Fix false positives for valueless attribute selectors in selector-attribute-name-disallowed-list #5060

merged 1 commit into from Dec 8, 2020

Conversation

vkrol
Copy link
Member

@vkrol vkrol commented Nov 23, 2020

Which issue, if any, is this issue related to?

Closes #5059.

Is there anything in the PR that needs further explanation?

No, it's self-explanatory.

/cc @rletsin

@jeddy3 jeddy3 changed the title Handle valueless attribute selectors in selector-attribute-name-disallowed-list rule Fix false positives for valueless attribute selectors in selector-attribute-name-disallowed-list Nov 24, 2020
Copy link
Member

@jeddy3 jeddy3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Copy link
Member

@mattxwang mattxwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🚀

@vkrol
Copy link
Member Author

vkrol commented Dec 8, 2020

Can I merge this PR myself?

@hudochenkov
Copy link
Member

@vkrol I'm not sure if you have rights to edit CHANGELOG via website https://stylelint.io/maintainer-guide/pull-requests

@vkrol
Copy link
Member Author

vkrol commented Dec 8, 2020

@hudochenkov thank you very much! I missed this guide :( I updated CHANGELOG.md.

@vkrol
Copy link
Member Author

vkrol commented Dec 8, 2020

@hudochenkov could you please review the change in CHANGELOG.md?

@hudochenkov
Copy link
Member

@vkrol I don't need to update changelog in a PR. After PR is merged we update changelog via webinterface. You might not have right to change changelog via web-interface, because it's a commit directly to master, and master is a protected branch.

Anyway, I'll merge this PR as it looks good :)

@hudochenkov hudochenkov merged commit 6b06b36 into stylelint:master Dec 8, 2020
@vkrol
Copy link
Member Author

vkrol commented Dec 9, 2020

@hudochenkov

After PR is merged we update changelog via webinterface.

Oh, sorry. This is not my best day ever :(

Thank you for your patience! :)

@vkrol vkrol deleted the handle-valueless-attribute-selectors-in-selector-attribute-name-disallowed-list-rule branch December 9, 2020 09:51
@hudochenkov
Copy link
Member

Don't worry! It's alright :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Fix false positives for valueless attribute selectors in selector-attribute-name-disallowed-list
4 participants