Skip to content

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