Skip to content

fix(core): skip vt100 parsing if tui disabled #31010

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 2 commits into from
May 2, 2025
Merged

Conversation

AgentEnder
Copy link
Member

Current Behavior

nx@21.0.0-rc.2 is failing on circle ci because it registers as a tty

Expected Behavior

We don't deal with vt100 unless in the tui, which doesn't happen on ci

Related Issue(s)

Fixes #

@AgentEnder AgentEnder requested review from a team as code owners May 2, 2025 16:36
@AgentEnder AgentEnder requested review from xiongemi and Cammisuli May 2, 2025 16:36
Copy link

vercel bot commented May 2, 2025

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 May 2, 2025 5:37pm

Copy link

nx-cloud bot commented May 2, 2025

View your CI Pipeline Execution ↗ for commit 0dc5585.

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

☁️ Nx Cloud last updated this comment at 2025-05-02 18:22:00 UTC

@AgentEnder AgentEnder force-pushed the fix/no-vt100-on-ci branch from c107097 to 8807309 Compare May 2, 2025 16:42
Copy link

github-actions bot commented May 2, 2025

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-31010-8807309 my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-31010-8807309
Release details 📑
Published version 0.0.0-pr-31010-8807309
Triggered by @AgentEnder
Branch fix/no-vt100-on-ci
Commit 8807309
Workflow run 14799380894

To request a new release for this pull request, mention someone from the Nx team or the @nrwl/nx-pipelines-reviewers.

Copy link

github-actions bot commented May 2, 2025

🐳 We have a release for that!

This PR has a release associated with it. You can try it out using this command:

npx create-nx-workspace@0.0.0-pr-31010-0dc5585 my-workspace

Or just copy this version and use it in your own command:

0.0.0-pr-31010-0dc5585
Release details 📑
Published version 0.0.0-pr-31010-0dc5585
Triggered by @AgentEnder
Branch fix/no-vt100-on-ci
Commit 0dc5585
Workflow run 14799380894

To request a new release for this pull request, mention someone from the Nx team or the @nrwl/nx-pipelines-reviewers.

@AgentEnder AgentEnder merged commit 6e1f304 into master May 2, 2025
6 checks passed
@AgentEnder AgentEnder deleted the fix/no-vt100-on-ci branch May 2, 2025 18:35
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