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

src: unset NODE_VERSION_IS_RELEASE #14005

Conversation

addaleax
Copy link
Member

When 53b8c51 was cherry-picked into master, NODE_VERSION_IS_RELEASE was accidentally set to 1 as well.

I’d suggest fast-tracking because it’s a trivial change and fixes the version reported by nightlies/builds off master.

When 53b8c51 was cherry-picked into
`master`, `NODE_VERSION_IS_RELEASE` was accidentally set to `1` as well.
@rvagg
Copy link
Member

rvagg commented Jun 30, 2017

herp derp, I could swear that I manually undid this!

lgtm

@rvagg
Copy link
Member

rvagg commented Jun 30, 2017

+1 to fast-track, get it done

@addaleax addaleax added dont-land-on-v6.x lib / src Issues and PRs related to general changes in the lib or src directory. labels Jun 30, 2017
@addaleax
Copy link
Member Author

Landed in a84c3be, thanks!

@addaleax addaleax closed this Jun 30, 2017
@addaleax addaleax deleted the this-is-not-the-release-you-are-looking-for branch June 30, 2017 12:36
addaleax added a commit that referenced this pull request Jun 30, 2017
When 53b8c51 was cherry-picked into
`master`, `NODE_VERSION_IS_RELEASE` was accidentally set to `1` as well.

PR-URL: #14005
Reviewed-By: Rod Vagg <rod@vagg.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lib / src Issues and PRs related to general changes in the lib or src directory.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants