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
[New] eslint ^5 support #451
Conversation
@@ -11,6 +11,8 @@ cache: | |||
yarn: true | |||
directories: | |||
- node_modules | |||
before_script: | |||
- if [ -n "${ESLINT-}" ]; then npm uninstall --no-save eslint-config-airbnb-base && npm install --no-save "eslint@${ESLINT}"; fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the uninstallation part can be removed once airbnb/javascript#1834 is complete, and an eslint v5-compatible eslint-config-airbnb-base is published.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@ljharb do you want to address your 'uninstallation' comment before merging?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this
@jessebeach it can't be addressed just yet; i'll clean it up in a followup once the base config is next published. |
@evcohen it'd be great if you could cut a release <3 (#452) |
I also improved the tests so we're testing on all supported eslint versions, and updated some deps.
There's currently one failing test on eslint 5; I'm working on that now.(linking back to airbnb/javascript#1834)