You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository was archived by the owner on Jan 19, 2019. It is now read-only.
Ah, it turns out this PR eslint/eslint#4604 was merged in to eslint-config-eslint. allowAfterThis (from no-underscore-dangle rule) is therefore being used in this project, but it is an ESLint 2.x only option.
Are you happy for me to submit a PR bumping eslint-config-eslint for this project to 2.0.0 and setting eslint to 2.0.0-beta.1?
You rightly said that this project won't be a priority until 2.x is stable anyway, so we can replace the beta with stable as soon as is possible, but in the meantime it would at least allow me to lint my PRs in a future-proof way...
Also, I usually work by setting exact dependency versions (no ^ or ~), and so that is what my npm install save behaviour is by default. Would you like me to do that or stick with the "looser" definitions in PRs which require dependency changes?
I have submitted a request to have eslint-config-eslint republished to npm to include the already-merged fix: eslint/eslint#4970
I will submit a PR with the version bump to close this issue, as and when it is available.
The text was updated successfully, but these errors were encountered: