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

deps: upgrade npm to 7.0.14 #36238

Closed
wants to merge 1 commit into from
Closed

deps: upgrade npm to 7.0.14 #36238

wants to merge 1 commit into from

Conversation

nlf
Copy link
Contributor

@nlf nlf commented Nov 23, 2020

7.0.14 (2020-11-23)

DEPENDENCIES

  • 09d21ab90 @npmcli/run-script@1.8.1
    • fix a regression in how scripts are escaped

@nodejs-github-bot nodejs-github-bot added the npm Issues and PRs related to the npm client dependency or the npm registry. label Nov 23, 2020
@ruyadorno ruyadorno added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 23, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 23, 2020
@nodejs-github-bot
Copy link
Collaborator

@MylesBorins MylesBorins added request-ci Add this label to start a Jenkins CI on a PR. fast-track PRs that do not need to wait for 48 hours to land. and removed request-ci Add this label to start a Jenkins CI on a PR. labels Nov 23, 2020
@MylesBorins
Copy link
Contributor

can we please fast-track?

@MylesBorins
Copy link
Contributor

@codebytere please include this in tomorrow's 15.x release. I'll work on making sure it is landed. Failed tests are due to master being broken

@MylesBorins
Copy link
Contributor

In an attempt to fix CI I've rebased out a couple of commits from master, fingers crossed that we can get a green CI

@MylesBorins MylesBorins added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 24, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 24, 2020
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@Trott
Copy link
Member

Trott commented Nov 24, 2020

Fix to CI issues has landed, so I'm running a fresh CI. Should be green or green-ish. I'll try to stay up until this lands. (It's not late here, but uh...yeah, I keep early hours these days.)

@nodejs-github-bot
Copy link
Collaborator

@MylesBorins
Copy link
Contributor

rebased and kicked off another CI, hopefully fixes broken actions

Copy link
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

codebytere pushed a commit that referenced this pull request Nov 24, 2020
PR-URL: #36238
Reviewed-By: Ruy Adorno <ruyadorno@github.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
@Trott
Copy link
Member

Trott commented Nov 24, 2020

Landed in 767070c

Trott pushed a commit to Trott/io.js that referenced this pull request Nov 24, 2020
PR-URL: nodejs#36238
Reviewed-By: Ruy Adorno <ruyadorno@github.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Shelley Vohr <codebytere@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Myles Borins <myles.borins@gmail.com>
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 48 hours to land. npm Issues and PRs related to the npm client dependency or the npm registry.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants