FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

src: unset `NODE_VERSION_IS_RELEASE` by addaleax · Pull Request #14005 · nodejs/node · GitHub

/ node Public

src: unset NODE_VERSION_IS_RELEASE - #14005

Closed
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:this-is-not-the-release-you-are-looking-for
Closed

src: unset NODE_VERSION_IS_RELEASE#14005
addaleax wants to merge 1 commit into
nodejs:masterfrom
addaleax:this-is-not-the-release-you-are-looking-for

Conversation

Copy link
Copy Markdown
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 commented Jun 30, 2017

Copy link
Copy Markdown
Member

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

lgtm

rvagg commented Jun 30, 2017

Copy link
Copy Markdown
Member

+1 to fast-track, get it done

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

Copy link
Copy Markdown
Member Author

Landed in a84c3be, thanks!

addaleax closed this Jun 30, 2017
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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL