-
Notifications
You must be signed in to change notification settings - Fork 51
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
Crashes with latest ESLint version (6.0.1
)
#118
Comments
Thank you for the report, I looked over the issue, it is easily fixable but if you don't mind I'd like to wait for a beta or rc release before working on it. |
Absolutely, no worries! Do you think I should report this bug to eslint as well? |
No, that's fine |
|
6.0.0-alpha.2
)6.0.0
)
|
Hey there! I lifted some ideas from this plugin when I was writing eslint-plugin-svelte3. I got that plugin updated for ESLint 6 today, so time to attempt to return the favor! Previously in eslint-plugin-svelte3, I was searching the require cache for the linter and monkey patching the ESLint 6 introduces a nice mechanism for named processors which aren't inherently associated to any file extension - and then the consumer can use Hope this is helpful! |
Still failing on |
6.0.0
)6.0.1
)
ESLint v6 support has been released with eslint-plugin-html v6 (I had to make a new major because ESLint did drop Node 6 support, so I decided to drop it as well). Let me know how it works for you. Cheers! @Conduitry this looks very interesting and definitely goes to the right direction, but I still have other constraints related to how script tags works in HTML (in particular, the common global scope) so I won't be able to send multiple files to ESLint. I'm glad it's helpful to you though! I may be able to simplify the patching logic though, I'll investigate when I have time. |
Thanks for the bug fix! I just tried it in all my repositories and it works. |
Description
Related to #117, but for the latest
eslint
version.When used together with the latest ESLint version (
6.0.0-alpha.2
),eslint
crashes.6.0.0-alpha.1
works.It's probably related to this PR and this issue.
.eslintrc.yml
:package.json
:package-lock.json
:file.md
:Then:
Setup configuration
6.0.0-alpha.2
5.0.5
12.3.1
19.04
The text was updated successfully, but these errors were encountered: