-
Notifications
You must be signed in to change notification settings - Fork 10
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
Updating major versions of dependencies #15
Updating major versions of dependencies #15
Conversation
"babel-eslint": "^4.1.7", | ||
"eslint-config-standard": "^4.4.0", | ||
"babel-eslint": "^5.0.0", | ||
"eslint-config-standard": "^5.1.0", | ||
"eslint-plugin-standard": "^1.3.1" |
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 plugin doesn't need updating?
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.
It's up to date - that's the latest
I updated the team Developers to have permissions for this repo, so the pull approve should work if you throw another commit on here. |
Cool, I wasn't sure how far you wanted to expand privileges |
Looks like I'm missing a new peer dependency anyway |
"babel-eslint": "^4.1.7", | ||
"eslint-config-standard": "^4.4.0", | ||
"babel-eslint": "^5.0.0", | ||
"eslint-config-standard": "^5.1.0", |
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.
Looks like this may require another dependency now
https://travis-ci.org/ciena-frost/eslint-config-frost-standard/builds/109890304
Error: Failed to load plugin promise: Cannot find module 'eslint-plugin-promise'
Good to go? |
Updating major versions of dependencies
This is a #major# change to move up the next versions of eslint and associated projects