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

Release/v7.5.4 #2684

Merged
merged 17 commits into from
Feb 12, 2021
Merged

Release/v7.5.4 #2684

merged 17 commits into from
Feb 12, 2021

Conversation

wraithgar
Copy link
Member

No description provided.

isaacs and others added 12 commits February 11, 2021 10:42
If an env script is already defined, run that
instead of the default.

PR-URL: #2655
Credit: @isaacs
Close: #2655
Reviewed-by: @ljharb
Emphasizing them like we do everywhere else to fix encoding errors and
for consistency.

Also removed the `<String>` from the type definition as we don't do that
anywhere else.

PR-URL: #2662
Credit: @ethomson
Close: #2662
Reviewed-by: @wraithgar
This brought in @npmcli/installed-package-contents@1.0.7 from
commit e1822cf
This pulls in, installs, and de-dupes our subdependencies.
Notable updates are promise-retry and @npmcli/move-file which
had new versions but we had no way to update and/or dedupe

We also manually removed uuid from our package.json which was
only added in the past to try to get around this same deduping
issue
This decouples our tests from depending on the node version being right, and
allows us to run in all of the environments we need to during CI.

PR-URL: #2681
Credit: @ljharb
Close: #2681
Reviewed-by: @wraithgar
@wraithgar wraithgar requested a review from a team as a code owner February 12, 2021 17:26
wraithgar and others added 3 commits February 12, 2021 09:29
In this node version, fall back to `util.promisify` of the callback version.

Maybe fixes #2623. Maybe fixes #2652. Maybe fixes #2625.

PR-URL: #2654
Credit: @ljharb
Close: #2654
Reviewed-by: @wraithgar, @ruyadorno
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants