Skip to content

fix(nx-cloud): allow download-cloud-client to work outside nx workspaces#34746

Merged
vsavkin merged 1 commit intomasterfrom
download-light-client-works-everywhere
Mar 6, 2026
Merged

fix(nx-cloud): allow download-cloud-client to work outside nx workspaces#34746
vsavkin merged 1 commit intomasterfrom
download-light-client-works-everywhere

Conversation

@vsavkin
Copy link
Copy Markdown
Member

@vsavkin vsavkin commented Mar 6, 2026

Remove the isNxCloudUsed guard that prevented the command from running without an nx.json. Now gracefully falls back to default cloud URL when not in an Nx workspace.

Current Behavior

Expected Behavior

Related Issue(s)

Fixes #

Remove the isNxCloudUsed guard that prevented the command from running
without an nx.json. Now gracefully falls back to default cloud URL
when not in an Nx workspace.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vsavkin vsavkin requested a review from a team as a code owner March 6, 2026 17:24
@vsavkin vsavkin requested a review from MaxKless March 6, 2026 17:24
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 6, 2026

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit a1e500d
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/69ab0dc010d6f10008605ea2
😎 Deploy Preview https://deploy-preview-34746--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 6, 2026

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit a1e500d
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/69ab0dbf047f750008b13e26
😎 Deploy Preview https://deploy-preview-34746--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Mar 6, 2026

View your CI Pipeline Execution ↗ for commit a1e500d

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 56m 40s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 3m 27s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 8s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-03-06 18:25:22 UTC

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 6, 2026

🐳 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@22.6.0-pr.34746.a1e500d my-workspace

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

22.6.0-pr.34746.a1e500d
Release details 📑
Published version 22.6.0-pr.34746.a1e500d
Triggered by @vsavkin
Branch download-light-client-works-everywhere
Commit a1e500d
Workflow run 22774348885

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

@vsavkin vsavkin merged commit d267145 into master Mar 6, 2026
23 checks passed
@vsavkin vsavkin deleted the download-light-client-works-everywhere branch March 6, 2026 20:18
MaxKless added a commit that referenced this pull request Mar 11, 2026
The isNxCloudCommand guard in nx.ts was missing download-cloud-client,
preventing it from running outside an Nx workspace despite #34746 fixing
the handler itself.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

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 Mar 12, 2026
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