Skip to content
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

fix(js): kill local registry child process if process exit #29356

Closed
wants to merge 1 commit into from

Conversation

xiongemi
Copy link
Collaborator

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

Copy link

vercel bot commented Dec 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Updated (UTC)
nx-dev ⬜️ Ignored (Inspect) Visit Preview Dec 16, 2024 3:56pm

Copy link

nx-cloud bot commented Dec 13, 2024

Your CI Pipeline Execution ↗ for commit b2591d0 ⏳ is in progress.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ❌ Failed 4m, 48s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m, 11s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 0s View ↗
nx-cloud record -- nx format:check --base=4cd64... ✅ Succeeded 28s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 27s View ↗
nx documentation --no-dte ✅ Succeeded 46s View ↗

Last updated at 2024-12-16 16:03:25UTC

Sent with 💌 from NxCloud.

@xiongemi xiongemi force-pushed the fix/kill-local-registry-process-exit branch from 67c4175 to 3311c08 Compare December 13, 2024 22:15
@xiongemi xiongemi marked this pull request as ready for review December 13, 2024 22:16
@xiongemi xiongemi requested review from meeroslav, vsavkin, mandarini and a team as code owners December 13, 2024 22:16
@xiongemi xiongemi force-pushed the fix/kill-local-registry-process-exit branch from 3311c08 to 537d6be Compare December 16, 2024 07:51
@xiongemi xiongemi force-pushed the fix/kill-local-registry-process-exit branch from 537d6be to b2591d0 Compare December 16, 2024 15:52
@xiongemi xiongemi closed this Dec 17, 2024
@njausteve
Copy link

@xiongemi whats the reason for closing this PR?

Copy link

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 23, 2024
@xiongemi xiongemi deleted the fix/kill-local-registry-process-exit branch February 19, 2025 08:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants