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
Set the node version in package.json #1793
Conversation
Testing against 0.10 and 0.12 was dropped in e8dbe6c
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.
good call. would you like to handle the release too?
I'm down. 👍 I know that whether or not dropping support for insecure versions of node should be a breaking change is a contentious topic, do you feel this should be a breaking or feature version bump? Personally I tend towards breaking releases if dropping support for versions of node that are in maintenance, and feature/patch for dropping support for versions of node that are out of LTS, but I don't have strong opinions. |
i think we should follow module-deps & detective and do a major bump :) |
Just make sure to update module-deps before doing the bump! |
The module-deps dependency got bumped in #1785, so it should be good to go. Do y'all prefer having pull requests opened for changelog additions, or is it fine if I push changelog+package.json straight into master when publishing? |
just pushing straight to master is how i've done it, so i'm cool w/ that! |
Testing against 0.10 and 0.12 was dropped in e8dbe6c