-
Notifications
You must be signed in to change notification settings - Fork 38
Fix the new recommended
ruleset
#6
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
Conversation
How did you encounter this problem? I've never seen such errors. |
@sodatea Here is a minimum reproducible example https://github.com/DavidRouyer/vue-cli-eslint My environment:
It failed on my Docker setup too so it's not related to Windows Maybe something is misconfigured? You can take a look at the commit https://github.com/DavidRouyer/vue-cli-eslint/commit/295a76950f93d3480372291a74bb71fd18ce2fc1 |
Reproduced it. Really weird that I did not encounter the error before. |
Thanks again! 💚 |
@sodatea The Airbnb config has an additional |
But somehow it passed our tests… 🤔 |
Are you sure you're testing both configurations? The |
We are testing against this eslintrc, which extends the The spec is here: https://github.com/vuejs/eslint-config-typescript/blob/master/test/index.spec.js |
Otherwise it fails with the following errors: