test: remove unnecessary status check on test-release-npm · nodejs/node@bd1eb14 · GitHub

nodejs / node Public

• Notifications You must be signed in to change notification settings
• Fork 36.6k
• Star 120k

• Code
• Issues 632
• Pull requests 755
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit bd1eb14

Browse filesBrowse the repository at this point in the historyBrowse files

RafaelGSS

authored and

MoLow

committed

test: remove unnecessary status check on test-release-npm

Signed-off-by: RafaelGSS <rafael.nunu@hotmail.com> PR-URL: #47516 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 parent 0560672 commit bd1eb14
Copy full SHA for bd1eb14

1 file changed

-1Lines changed: 0 additions & 1 deletion

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• test/parallel
  • test-release-npm.js

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎test/parallel/test-release-npm.js‎

Copy file name to clipboardExpand all lines: test/parallel/test-release-npm.js

-1Lines changed: 0 additions & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -17,7 +17,6 @@ if (!releaseReg.test(process.version) || !common.hasCrypto) {
17	17
18	18	const npmCli = path.join(__dirname, '../../deps/npm/bin/npm-cli.js');
19	19	const npmExec = child_process.spawnSync(process.execPath, [npmCli]);
20		- assert.strictEqual(npmExec.status, 1);
21	20
22	21	const stderr = npmExec.stderr.toString();
23	22	assert.strictEqual(stderr.length, 0, 'npm is not ready for this release ' +

0 commit comments

Comments

0 (0)