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
Drop support for Node.js < 4 #11
Conversation
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 very much! I agree with the changes, just please keep the versions pinned.
- 4 | ||
- 6 | ||
- 8 | ||
- 'stable' |
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.
Ha! This is good. We could be using it also with the Dredd repo.
package.json
Outdated
"chai": "1.7.1" | ||
"chai": "^4.1.2", | ||
"coffeescript": "^1.12.7", | ||
"mocha": "^4.0.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.
There's a new process with us which requires us to use only approved dependencies. That basically means the deps' licensing needs to be all right. For this reason we're not relaxing the versions anymore.
To be honest, I wasn't much fan of the relaxed versions anyway. To me as a maintainer, they bring more hassle than benefit and it felt to me it's always better to just explicitly upgrade the deps, e.g. with the help of https://greenkeeper.io/
Would you please keep the versions pinned?
Updated with pinned versions. Sorry it took so long. Setting up GreenKeeper sounds like a good idea with this new constraint. |
package.json
Outdated
"chai": "1.7.1" | ||
"chai": "4.1.2", | ||
"coffeescript": "1.12.7", | ||
"mocha": "5.0.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.
I'm sorry this takes so long. We need to get this kumavis/browser-stdout#4 resolved first. Current Mocha has a license issue - their dependency tree contains the browser-stdout
library, which has invalid license. I'll get back to this as soon as possible.
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.
Filed mochajs/mocha#3248
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.
Ah, that's a bummer. Looks like that dependency was added this commit which was first part of version 3.0.
Would it help if I used mocha 2.5.3 instead? That's the last release before this dependency was added.
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.
There is some activity on the Mocha issue, so let's see how it turns out. I'd better upgrade than downgrade, if possible. The same problem hurts Dredd, etc.
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.
browser-stdout
1.3.1 was released - now with a license file 🎉
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.
Also updates dev dependencies to their latest version.