-
Notifications
You must be signed in to change notification settings - Fork 26
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
Conflict with eslint rule "spaced-comment" #67
Comments
Hi @ClementMindflow Sorry for being late. html-eslint does not work with ECMAScript AST, It uses a AST made from
I think It would be great if we add a new |
it seems that any rules that has to do with comments cause troubles. I have a similar issue with the |
I'm facing the same issue as @ClementMindflow. Please let me know if there any plans on incorporating the fix or workarounds available? |
Fixed it temporarily by turning off the 'space-comments' rule for *.html alone in overrides key. Is there a way to permanently fix this? |
@FatehAK For permanently fixing it we should create a new rule (ie: |
This bug still exists today. |
When a HTML comment is parsed using
html-eslint
it produce comment node of typecomment
.If that node is given to the eslint builtin
spaced-comment
rule, it will fail with the following error:After a quick look at the code, it seems like the
spaced-comment
rule expect comment node to have thetype
attribute set to eitherblock
orline
, and nodecomment
( see here )Maybe setting the node type to the appropriate type would fix the issue ...
The text was updated successfully, but these errors were encountered: