Skip to content

Conversation

@ntotten
Copy link
Member

@ntotten ntotten commented Aug 6, 2025

No description provided.

Copilot AI review requested due to automatic review settings August 6, 2025 18:29
@vercel
Copy link

vercel bot commented Aug 6, 2025

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

Name Status Preview Updated (UTC)
zudoku-cosmo-cargo ✅ Ready (Inspect) Visit Preview Aug 6, 2025 6:31pm
zudoku-docs ✅ Ready (Inspect) Visit Preview Aug 6, 2025 6:31pm

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes Nx Cloud integration from the project by disabling cloud connectivity and removing cloud access tokens from CI workflows.

  • Replaces the Nx Cloud ID configuration with a directive to never connect to the cloud
  • Removes Nx Cloud access token environment variables from GitHub Actions workflows

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
nx.json Disables Nx Cloud connection by replacing cloud ID with neverConnectToCloud: true
.github/workflows/main.yaml Removes NX_CLOUD_ACCESS_TOKEN environment variable
.github/workflows/branch-prerelease.yaml Removes NX_CLOUD_ACCESS_TOKEN environment variable

@github-actions
Copy link

github-actions bot commented Aug 6, 2025

Preview build of published Zudoku package for commit 43a1eff.

See the deployment at: https://6f02f4fd.cosmocargo-public-package.pages.dev

Note

This is a preview of the Cosmo Cargo example using the Zudoku package published to a local registry to ensure it'll be working when published to the public NPM registry.

Last updated: 2025-08-06T18:32:33.589Z

@ntotten ntotten merged commit 9df984c into main Aug 6, 2025
9 checks passed
@ntotten ntotten deleted the remove_nx_cache branch August 6, 2025 19:01
@zuplo-review-bot
Copy link

🔴 Pull request merged without review. #6831

Copy link
Contributor

@lcampos lcampos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed and approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants