Skip to content

fix(core): tui summary should handle in progress tasks properly #30905

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

Merged
merged 1 commit into from
Apr 29, 2025

Conversation

AgentEnder
Copy link
Member

Current Behavior

When running via run-many failures that include a continuous task are indicated as a cancellation on the tui summary

Expected Behavior

Failures that run to completion are not cancelled, and should be displayed as a failure

Related Issue(s)

Fixes #

@AgentEnder AgentEnder requested a review from a team as a code owner April 28, 2025 20:41
@AgentEnder AgentEnder requested a review from FrozenPandaz April 28, 2025 20:41
Copy link

vercel bot commented Apr 28, 2025

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Apr 28, 2025 11:04pm

Copy link

nx-cloud bot commented Apr 28, 2025

View your CI Pipeline Execution ↗ for commit cdc7366.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-ci ✅ Succeeded 44m 28s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 18s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 2m 11s View ↗

☁️ Nx Cloud last updated this comment at 2025-04-28 23:33:23 UTC

@FrozenPandaz FrozenPandaz merged commit 4b9eab9 into master Apr 29, 2025
6 checks passed
@FrozenPandaz FrozenPandaz deleted the fix/tui-summary-in-progress branch April 29, 2025 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants