Skip to content

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

Rodrigo Muino Tomonari requested to merge github/fork/RafaelGSS/fix/test-release-npm into main

test-release-npm is failing on #47381 and it seems related to https://github.com/nodejs/node/pull/43716. However, this asserting seems unnecesary, so rather than adjust it (either on npm side or node.js) sounds reasonable to remove that check.

However, it doesn't mean there's no bug in that change. It seems process.exitCode isn't considered when process.exit() is called with undefined. cc: @daeyeon

Merge request reports