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

Tap dependency update to v11 #350

Closed
exsilium opened this issue Feb 20, 2018 · 9 comments
Closed

Tap dependency update to v11 #350

exsilium opened this issue Feb 20, 2018 · 9 comments

Comments

@exsilium
Copy link

Can we have nodeunit pump the tap dep to v11 please? Thank you! 🙇

@mreinstein
Copy link
Collaborator

mreinstein commented Feb 20, 2018

unfortunately this isn't possible at the moment. nodeunit supports really, really old versions of node. tap v11 dropped support for older versions.

@exsilium
Copy link
Author

Thanks @mreinstein! But what's the strategy here going forward from nodeunit perspective? I'd expect people stuck in legacy non-supported node versions are ok to lock themselves to a particular version of nodeunit as well. At some point the drop needs to happen, or not? Cheers! 🍻

@andy-ganchrow
Copy link

In the dependency tree of tap v10.7.3 is hoek v2.16.3 which apparently has a security vulnerability, so it would be great if this were resolved.

@mreinstein
Copy link
Collaborator

PR welcome!

@mreinstein
Copy link
Collaborator

something to note: nodeunit is essentially a deprecated testing library. There are a lot of modern alternatives that are better at this point.

@brettz9
Copy link
Contributor

brettz9 commented May 10, 2018

A warning to that effect would be very helpful!

@mreinstein
Copy link
Collaborator

A warning to that effect would be very helpful!

Yes, Caolan and I have been meaning to do that for a while. Finally done via ec2ea88, thanks for the reminder. 👍

@brodycj
Copy link
Contributor

brodycj commented Jul 9, 2018

FYI update to tap 12 in PR #356 seems to pass old Node.js versions back to 0.12, though with a few ugly engine warnings.

@mreinstein
Copy link
Collaborator

closing since tap has moved on from v11

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

No branches or pull requests

5 participants