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

v7.24.2 #3818

Closed
wants to merge 14 commits into from
Closed

v7.24.2 #3818

wants to merge 14 commits into from

Conversation

lukekarrys
Copy link
Member

v7.24.2 (2021-09-30)

BUG FIXES

DEPENDENCIES

  • 69ab10bbf is-core-module@2.7.0
  • e94ddeaca @npmcli/arborist@2.9.0:
    • fix: avoid infinite loops in peer dep replacements
    • fix: use Intl.Collator for string sorting when available
    • feat(vuln): expose isDirect
  • 8349c3c15 #3815 arborist@2.10.0:
    • includeWorkspaceRoot support
    • workspacesEnabled=false support

DOCUMENTATION

wraithgar and others added 14 commits September 27, 2021 09:31
will filter out a small subset of non-URL-safe characters that still
parse properly with `new URL`

PR-URL: #3804
Credit: @isaacs
Close: #3804
Reviewed-by: @wraithgar
closes: #2556
xref: #1750

The xref'ed PR apparently dropped this behavior
without any explanation.

PR-URL: #3799
Credit: @gfyoung
Close: #3799
Reviewed-by: @wraithgar
  * fix: avoid infinite loops in peer dep replacements
  * fix: use Intl.Collator for string sorting when available
  * feat(vuln): expose isDirect
The npm/cli form of npm/arborist#324

Required adding options support to package used for this.

PR-URL: #3809
Credit: @isaacs
Close: #3809
Reviewed-by: @wraithgar
  * includeWorkspaceRoot support
  * workspacesEnabled=false support

PR-URL: #3815
Credit: @wraithgar
Close: #3815
Reviewed-by: @isaacs
Adds a new config item that includes the workspace root when running
non-arborist commands (i.e. repo, version, publish).  Arborist will need
to be udpated to look for this flag to change its behavior to include
the workspace root for its functions.

This also changes --workspaces to a trinary, so that setting it to false
will explicitly exclude workspaces altogether.  This is also going to
require an arborist change so that it ignores workspaces altogether.

Co-author: @fritzy

PR-URL: #3816
Credit: @isaacs
Close: #3816
Reviewed-by: @wraithgar
This reverts commit 8349c3c.

PR-URL: #3817
Credit: @lukekarrys
Close: #3817
Reviewed-by: @wraithgar
@lukekarrys lukekarrys requested a review from a team as a code owner September 30, 2021 21:15
@lukekarrys lukekarrys closed this Sep 30, 2021
@lukekarrys lukekarrys deleted the release/v7.24.2 branch October 4, 2021 16:54
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.

None yet

5 participants