Skip to content

Conversation

@mw-w
Copy link
Contributor

@mw-w mw-w commented Oct 28, 2025

Summary

Remove nx access token and use access token in github secrets, environment variable supersedes nx.json

Detail and impact of the change

Added

  • NX Access Token secret variable in workflows that use NX

Removed

  • NX Access Token from nx.json

Security

  • Rotate NX access token after merge

@mw-w mw-w requested review from a team as code owners October 28, 2025 03:39
@nx-cloud
Copy link

nx-cloud bot commented Oct 28, 2025

View your CI Pipeline Execution ↗ for commit f82d2ae

Command Status Duration Result
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 4s View ↗

☁️ Nx Cloud last updated this comment at 2025-10-28 23:27:24 UTC

Copy link
Contributor

@timm088 timm088 left a comment

Choose a reason for hiding this comment

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

👍 Good with this, matches NX best practices.

@mw-w mw-w added this pull request to the merge queue Dec 3, 2025
Merged via the queue into main with commit 2f409fc Dec 3, 2025
11 of 12 checks passed
@mw-w mw-w deleted the task/move-nx-access-token-to-github-secrets branch December 3, 2025 02:38
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.

4 participants