-
Notifications
You must be signed in to change notification settings - Fork 140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test: fix runtime ci #1932
test: fix runtime ci #1932
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
4538cbd
to
f42b963
Compare
packages/runtime/lib/api.js
Outdated
@@ -43,16 +45,25 @@ class RuntimeApi { | |||
|
|||
setGlobalDispatcher(globalDispatcher) | |||
this.#dispatcher = globalDispatcher | |||
|
|||
process.on('SIGINT', async () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be moved to the top-level worker file.
@@ -7,7 +7,7 @@ | |||
"plt-runtime": "./runtime.mjs" | |||
}, | |||
"scripts": { | |||
"test": "npm run lint && node ./test/runner.js test/cli/*.test.mjs && tsd", | |||
"test": "npm run lint && node ./test/runner.js && tsd", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add back the cli
stuff?
packages/runtime/lib/api.js
Outdated
parentPort.postMessage(res) | ||
|
||
if (command === 'plt:stop-services') { | ||
console.log('terminate worker thread') | ||
process.exit() // Exit the worker thread. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this process.exit
hould be moved to the top-level worker file
9b18886
to
272bcf8
Compare
272bcf8
to
d8187aa
Compare
9c2d2eb
to
7d91f37
Compare
0d81f95
to
43057f7
Compare
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Updated and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: pino@8.16.2, pino-pretty@10.2.3, tap@16.3.9 |
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
Signed-off-by: Matteo Collina <hello@matteocollina.com>
No description provided.