Skip to content

src: unset NODE_VERSION_IS_RELEASE from master#30584

Closed
targos wants to merge 1 commit intonodejs:masterfrom
targos:is-release
Closed

src: unset NODE_VERSION_IS_RELEASE from master#30584
targos wants to merge 1 commit intonodejs:masterfrom
targos:is-release

Conversation

@targos
Copy link
Copy Markdown
Member

@targos targos commented Nov 22, 2019

No description provided.

@targos targos added the fast-track PRs that do not need to wait for 72 hours to land. label Nov 22, 2019
@targos
Copy link
Copy Markdown
Member Author

targos commented Nov 22, 2019

This fixes CI, let's fast-track!

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@gireeshpunathil
Copy link
Copy Markdown
Member

This fixes CI

@targos - curious what issues this was causing in the CI? I have been running and seeing a lot of failures in the CI for the C&L PRs, just want to know are those related some how.

@targos
Copy link
Copy Markdown
Member Author

targos commented Nov 22, 2019

@gireeshpunathil When that bit is set, test/parallel/test-release-npm is not skipped. On master, that test fails because npm warns that it's not compatible with Node 14.

@MylesBorins
Copy link
Copy Markdown
Contributor

apologies for missing this

Copy link
Copy Markdown
Contributor

@MylesBorins MylesBorins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MylesBorins
Copy link
Copy Markdown
Contributor

There is a backlog on CI, this is definitely not going to break anything. I think we should land this immediately and skip waiting for CI.

@gireeshpunathil
Copy link
Copy Markdown
Member

+1 for landing without waiting for CI - saves a lot of runs.

@MylesBorins
Copy link
Copy Markdown
Contributor

Landed in 917fedd

MylesBorins pushed a commit that referenced this pull request Nov 22, 2019
PR-URL: #30584
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
@targos targos deleted the is-release branch November 22, 2019 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track PRs that do not need to wait for 72 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants