-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
eslint-disable
comments in svelte html don't work with eslint-plugin-eslint-comments
#328
Comments
Thank you for posting the issue. eslint-disable HTML comments provided by eslint-plugin-svelte are completely different from eslint-disable comments provided by ESLint. |
I'm not using esling-plugin-eslint-comments, but I'm finding that: <!-- eslint-disable-next-line svelte/no-at-html-tags -->
{@html foobar} doesn't work either. Is that expected? Is there another way to ignore rules from this plugin on specific lines? (This is one particular case where I do want the tag, and it's guarded by a check that it's been stripped of anything potentially malicious.) |
@OJFord that works for me. could you share some more info like your eslint config or maybe some of the surrounding code? |
Hello! Just a note that I have tried this out, with a different rule from @intlify and it seems to be working for me, the rule is disabled as expected:
|
Before You File a Bug Report Please Confirm You Have Done The Following...
What version of ESLint are you using?
8.30.0
What version of
eslint-plugin-svelte
are you using?2.14.1
What did you do?
Configuration
What did you expect to happen?
eslint-comments/require-description
error on the html commentWhat actually happened?
no error on the html comment
Link to GitHub Repo with Minimal Reproducible Example
https://github.com/DetachHead/eslint-plugin-svelte-issue/tree/d0eb35b9d72cd26928488f7bbe8cb0ee0f0d1222
Additional comments
i'm not sure whether this is an issue with this plugin or
eslint-plugin-eslint-comments
, so i'm raising it in both projectsmysticatea/eslint-plugin-eslint-comments#74
The text was updated successfully, but these errors were encountered: