test: use common.crashOnUnhandledRejection#17215
test: use common.crashOnUnhandledRejection#17215sj82516 wants to merge 1 commit intonodejs:masterfrom
Conversation
There was a problem hiding this comment.
I think common needs to be declared first:
const common = require('../common');|
@sj82516 - your second commit brought about 94 files - accidental? |
|
I run |
|
It seems a bunch of pre-existing node modules are now being removed? |
|
yes, I removed them. |
Removing my sign off until the extraneous commits can be cleaned up
jasnell
left a comment
There was a problem hiding this comment.
This PR will need to be reworked to remove the extraneous edits
1487903 to
449be86
Compare
|
I rebase the commit and push force again. |
|
Still the same problem. I think you will need to edit your commit since it seems the file removals are a part of the same commit as your actual (test) changes. |
reset the commit to remvoe extra instructions.
|
I see. |
|
@sj82516 you can always force push over the branch you made the PR with |
449be86 to
d47c150
Compare
|
Looks better now. |
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
|
landed in 7518617 with some minor nits addressed |
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
PR-URL: #17215 Reviewed-By: Myles Borins <myles.borins@gmail.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
Add common.crashOnUnhandledRejection to test/parallel/test-microtask-queue-integration-domain.js
Checklist
make -j4 test(UNIX), orvcbuild test(Windows) passesAffected core subsystem(s)