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

Drop support for Node.js < 4 #11

Merged
merged 4 commits into from Mar 6, 2018
Merged

Drop support for Node.js < 4 #11

merged 4 commits into from Mar 6, 2018

Conversation

realityking
Copy link
Contributor

Also updates dev dependencies to their latest version.

Copy link
Contributor

@honzajavorek honzajavorek left a 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'
Copy link
Contributor

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"
Copy link
Contributor

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?

@realityking
Copy link
Contributor Author

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"
Copy link
Contributor

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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor Author

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.

Copy link
Contributor

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.

Copy link
Contributor Author

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 🎉

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@honzajavorek honzajavorek merged commit 037c11c into apiaryio:master Mar 6, 2018
@realityking realityking deleted the node-4 branch March 6, 2018 16:10
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

Successfully merging this pull request may close these issues.

None yet

2 participants