Skip to content

Commit 3eb54f8

Browse files
committed
refactor: show npm log in case of error
1 parent 14f4776 commit 3eb54f8

1 file changed

Lines changed: 12 additions & 3 deletions

File tree

lib/package.js

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -397,10 +397,12 @@ export default class Package {
397397

398398
const res = childProcess.spawnSync( "npm", params, {
399399
"shell": true,
400-
"stdio": "inherit",
400+
"stdio": "pipe",
401401
} );
402402

403403
if ( res.status ) {
404+
console.log( res.stderr + "" );
405+
404406
console.log( `Unable to publish to the npm registry` );
405407

406408
return result( 500 );
@@ -419,10 +421,15 @@ export default class Package {
419421
const res = await repeatAction( async () => {
420422
console.log( "Adding tag:", "npm", params.join( " " ) );
421423

422-
const res = childProcess.spawnSync( "npm", params, { "shell": true, "stdio": "inherit" } );
424+
const res = childProcess.spawnSync( "npm", params, {
425+
"shell": true,
426+
"stdio": "pipe",
427+
} );
423428

424429
// error
425430
if ( res.status ) {
431+
console.log( res.stderr + "" );
432+
426433
console.log( `Unable to add tag "${ tag }"` );
427434

428435
return result( 500 );
@@ -1015,10 +1022,12 @@ export default class Package {
10151022
else {
10161023
res = childProcess.spawnSync( "npm", [ "test" ], {
10171024
"cwd": this.root,
1018-
"stdio": "inherit",
1025+
"stdio": "pipe",
10191026
} );
10201027

10211028
if ( res.status ) {
1029+
console.log( res.stderr + "" );
1030+
10221031
res = result( [ 500, "Tests failed" ] );
10231032
}
10241033
else {

0 commit comments

Comments
 (0)