npm/cli Pull Requests

Review 103 pull requests

Pull request opened by github-actions[bot]

Merged

feat(token): support read-write-stage-only granular access tokens

	0.0 days	Authoring commits before pull request opened for review
	0.0 days	Awaiting first review
	2.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 0.0 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 0.0 business hours

Dhei Approved

#9994 with 1 commit opened 4 days ago by github-actions[bot]

backport/v11/9948

Opened and closed September 16

Review Code View on GitHub

16

Diff Delta

484 under target

0

comments

10 under target

Open:

fix(arborist): preserve `bundleDependencies: true` when saving package.json

Opened by Oliver Byford 11 days ago

Aug 31

No reviewer comments found yet.

#9958 with 2 commits opened 11 days ago by 36degrees

preserve-bundle-dependencies-true

Next action:

Code review

Review code

View at GitHub

0 pull request requested reviewers:

No reviewers nor requested reviewers found.

Last action:

Sep 9 by 36degrees

Open:

feat(token): support read-write-stage-only granular access tokens

Opened by Tayvon Wright 17 days ago

Aug 31

Last activity: Sep 3 by ljharb

1 reviewer

awaited review for 0.0 business hours

Jordan Harband

#9948 with 2 commits opened 17 days ago by Tayvon

tayvon-stage-only-gat-parity

Next action:

1 unresolved comment

Review code

View at GitHub

1 pull request reviewer:

Last action:

Sep 3 by ljharb

Merged

fix: provenance-file takes precedence over OIDC auto-generated provenance

	0.0 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.0 days	Merge to deploy

Total time for pull request 0.3 business days from first commit authored to deploy

1 reviewer

awaited review for 3.1 business hours

martin Approved

#9947 with 1 commit opened 17 days ago by github-actions[bot]

backport/v11/9882

Opened and closed September 3

Review Code View on GitHub

86

Diff Delta

415 under target

0

comments

10 under target

Merged

fix(config): avoid exporting persistent allow-scripts

N/A	6.0 days	Issue creation to work beginning
	0.0 days	Authoring commits before pull request opened for review
	1.8 days	Awaiting first review
	12.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 1.8 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 16.3 business hours

martin Approved

#9937 with 1 commit opened 20 days ago by github-actions[bot]

backport/v11/9913

Open August 31 - September 2

Review Code View on GitHub

[BUG] A user/local .npmrc allow-scripts setting is forwarded to an inner npm install spawned by npm run-script and fails with EALLOWSCRIPTS

0

Diff Delta

500 under target

0

comments

10 under target

Merged

fix(arborist): reject uninstall args that carry a version

N/A	7.0 days	Issue creation to work beginning
	2.2 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	0.0 days	Merge to deploy

Total time for pull request 2.9 business days from first commit authored to deploy

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9930 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9881

Open August 27 - August 28

Review Code View on GitHub

[BUG] `npm uninstall <name>@<version>` neither executes nor fails

0

Diff Delta

500 under target

0

comments

10 under target

Merged

fix(arborist): match allowScripts keys for local paths

N/A	3.9 days	Issue creation to work beginning
	2.0 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	15.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 2.7 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9929 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9914

Open August 27 - August 28

Review Code View on GitHub

[BUG] allowScripts does not work for local tarball on Windows

0

Diff Delta

500 under target

0

comments

10 under target

Merged

fix: keep dry-run output valid json

N/A	247.0 days	Issue creation to work beginning
	2.0 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	15.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 2.7 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9928 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9823

Open August 27 - August 28

Review Code View on GitHub

[BUG] npm install --json no longer JSON

0

Diff Delta

500 under target

0

comments

10 under target

Merged

fix: don't print the funding message for global installs

N/A	3.0 days	Issue creation to work beginning
	11.0 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	15.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 11.7 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9927 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9864

Open August 27 - August 28

Review Code View on GitHub

[BUG] `npm fund` reports no packages, only one username

0

Diff Delta

500 under target

0

comments

10 under target

Merged

chore(arborist): add missing registry mock in bin links reify test

	34.1 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	0.0 days	Merge to deploy

Total time for pull request 34.8 business days from first commit authored to deploy

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9926 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9746

Open August 27 - August 28

Review Code View on GitHub

0

Diff Delta

500 under target

0

comments

10 under target

Merged

chore: recognize prefixed Node.js PR titles

	1.6 days	Authoring commits before pull request opened for review
	0.3 days	Awaiting first review
	0.4 days	Revising work while processing 1 reviewer who left 0 comments
	15.0 days	Merge to deploy (still awaiting deployment)

Total time for pull request 2.3 business days from first commit authored to merge
Still awaiting deployment

1 reviewer

awaited review for 2.7 business hours

reggi Approved

#9925 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9915

Open August 27 - August 28

Review Code View on GitHub

0

Diff Delta

500 under target

0

comments

10 under target

Open:

chore: release 11.20.0

Opened by github-actions[bot] 24 days ago

Aug 31

No reviewer comments found yet.

#9922 with 1 commit opened 24 days ago by github-actions[bot]

release-please--branches--release/v11

Next action:

Code review

Review code

View at GitHub

0 pull request requested reviewers:

No reviewers nor requested reviewers found.

Last action:

Aug 27 by github-actions[bot]

Merged

feat(stage): display staged package status

	2.8 days	Authoring commits before pull request opened for review
	0.1 days	Awaiting first review
	0.2 days	Revising work while processing 2 reviewers who left 0 comments
	0.0 days	Merge to deploy

Total time for pull request 3.1 business days from first commit authored to deploy

2 reviewers

awaited review for 0.8 business hours

Joel Verhagen Approved
reggi Approved

#9920 with 1 commit opened 24 days ago by github-actions[bot]

backport/v11/9898

Opened and closed August 27

Review Code View on GitHub

10

Diff Delta

490 under target

0

comments

10 under target

Merged

chore: release 11.19.1

	0.0 days	Authoring commits before pull request opened for review
	1.1 days	Awaiting first review
	1.0 day	Merge to deploy

Total time for pull request 2.1 business days from first commit authored to deploy

1 reviewer

awaited review for 9.6 business hours

reggi Approved

#9917 with 1 commit opened 26 days ago by github-actions[bot]

release-please--branches--release/v11

Open August 25 - August 26

Review Code View on GitHub

0

Diff Delta

500 under target

3

comments

7 under target

Closed

fix: exempt explicit pack targets from allow-directory

N/A	31 days	Issue creation to work beginning
	0 days	Authoring commits before pull request opened for review
	1 day	Awaiting first review
	0 days	Revising work while processing 1 reviewer who left 0 comments

Total time for pull request 1 business day from first commit to closed PR

1 reviewer

awaited review for 6.1 business hours

martin Approved

#9916 with 1 commit opened 27 days ago by github-actions[bot]

backport/v11/9756

Open August 24 - August 25

Review Code View on GitHub

[BUG] `allow-directory=none|root` breaks npm publish

0

Diff Delta

500 under target

0

comments