-
Notifications
You must be signed in to change notification settings - Fork 22
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
rule 'attribute-names' is also applied to internal reactive state #202
Comments
good catch. seems like a bug for sure contributions welcome, but if not, ill try take a look soon 👍 |
I'll see if I can get around to taking a look at it this weekend. If I do, I'll be happy to post a pull request. |
its probably as simple as changing this condition to also check for eslint-plugin-lit/src/rules/attribute-names.ts Lines 40 to 42 in fb4dcfe
but i'd start by adding a test with similar code to your example and go from there |
I have opened a pullrequest. Would you be so kind to review it? |
Fixed by #203 |
In the following class, the
attribute-names
” rule is applied to theisFoo
property, although it is decorated with@state
and therefore has no public attribute.The rule is applied as well if the static properties syntax is used.
The text was updated successfully, but these errors were encountered: