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.

Sign up for GitHub

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

Jump to bottom

fix(core): do not warn about custom tasks runners when explicitly using default #29663

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix(core): do not warn about custom tasks runners when explicitly using default #29663

wants to merge 2 commits into from
+8 −4

Conversation

AgentEnder
Copy link
Member

Current Behavior

When default tasks runner is explicitly enabled a warning is shown.

Expected Behavior

No warning

Related Issue(s)

Fixes #

@AgentEnder AgentEnder requested a review from a team as a code owner January 17, 2025 15:55
@nx-cloud Nx Cloud
Copy link

nx-cloud bot commented Jan 17, 2025 •
edited
Loading

View your CI Pipeline Execution ↗ for commit 9e6704b.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ❌ Failed 21m 50s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 22s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check --base=a468d... ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation --no-dte ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-01-18 15:07:23 UTC

@vercel Vercel
Copy link

vercel bot commented Jan 17, 2025 •
edited
Loading

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

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Jan 17, 2025 11:22pm

@AgentEnder AgentEnder requested a review from a team as a code owner January 17, 2025 23:11
@AgentEnder AgentEnder requested a review from Cammisuli January 17, 2025 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FrozenPandaz FrozenPandaz FrozenPandaz approved these changes

@Cammisuli Cammisuli Cammisuli approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AgentEnder @Cammisuli @FrozenPandaz