Skip to content
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

.eslintrc camelcase configuration error #84

Closed
Taenick opened this issue Feb 22, 2019 · 6 comments
Closed

.eslintrc camelcase configuration error #84

Taenick opened this issue Feb 22, 2019 · 6 comments

Comments

@Taenick
Copy link

Taenick commented Feb 22, 2019

I currently have the following camelcase rule in my .eslintrc file:

"camelcase": ["error", { "allow": ["^UNSAFE_"] }]

According to the eslint documentation, this is completely valid: https://eslint.org/docs/rules/camelcase#allow

But when I run es6plato I get the following error:

Error: .eslintrc: Configuration for rule "camelcase" is invalid: Value {"allow":["^UNSAFE_"]} should NOT have additional properties.

Is this intended to happen?

@the-simian
Copy link
Owner

this might be related to the eslint version, you don't by chance have an older one in the project you're using this with?

see this thread comment: eslint/eslint#10783 (comment)

This change was in august, too so it might be the version of eslint I'm using too, in which case I can just update the dep.

@Taenick
Copy link
Author

Taenick commented Feb 25, 2019

That project is using eslint 5.14.1, which is the latest version according to the npm package site: https://www.npmjs.com/package/eslint

If that feature was just added in august then maybe updating the eslint in the es6plato module will resolve this.

Let me know if there is anything else I should look at.

@the-simian
Copy link
Owner

@Taenick thanks for the reply, I think you're right lets try that

@Taenick
Copy link
Author

Taenick commented Mar 12, 2019

@the-simian Any chance that you can merge the eslint update pr in? #83

@the-simian
Copy link
Owner

@Taenick this should be update in 1.1.6

@Taenick
Copy link
Author

Taenick commented Mar 19, 2019

@the-simian Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants