node
node copied to clipboard
test: change promise to async/await in test-debugger-help.mjs
Changes:
- Updated the test-debugger-help.js to test-debugger-help.mjs
- Updated the test-debugger-help to use async/await instead of a promise.
Welcome @chanduMe and thanks for the pull request. Can you please update the commit message to conform with our requirements? The first line needs to be 5 characters shorter so may this?:
test: use async/await in test-debugger-help
CI: https://ci.nodejs.org/job/node-test-pull-request/46663/
CI: https://ci.nodejs.org/job/node-test-pull-request/46776/
CI: https://ci.nodejs.org/job/node-test-pull-request/46788/
CI: https://ci.nodejs.org/job/node-test-pull-request/46852/
CI: https://ci.nodejs.org/job/node-test-pull-request/46914/
CI: https://ci.nodejs.org/job/node-test-pull-request/46938/
I'm not sure why the macOS test system keeps failing on Jenkins, but I'm going to rebase and force-push in the hope that there's a fix in the code base somewhere that we just need to include here.
I'm not sure why the macOS test system keeps failing on Jenkins, but I'm going to rebase and force-push in the hope that there's a fix in the code base somewhere that we just need to include here.
sequential/test-watch-mode-inspect failed. It's not as obvious in Jenkins anymore because the Node.js tap runner means we end up with tap embedded within tap which requires fixes in https://github.com/nodejs/tap2junit -- I'm waiting for a new version to be released (https://github.com/nodejs/tap2junit/issues/41) so I can (easily) deploy the update onto the test infrastructure.
CI: https://ci.nodejs.org/job/node-test-pull-request/46943/
CI: https://ci.nodejs.org/job/node-test-pull-request/46949/
Landed in e236af8d54495c2c358344ba56dd0f7408fece99
Thanks for the contribution! 🎉