-
Notifications
You must be signed in to change notification settings - Fork 38
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
chore: setup Nx Cloud Id for PAT support #2410
Conversation
e556113
to
e35aea5
Compare
e35aea5
to
afe64d7
Compare
|
||
jobs: | ||
# Check the integrity of yarn lock | ||
yarn_lock_check: | ||
runs-on: ubuntu-latest | ||
env: | ||
NX_SKIP_NX_CACHE: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not related to your change but don't we want the Nx cache for this step?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is required by nx-cloud
runner when run in CI.
cf: https://github.com/AmadeusITGroup/otter/actions/runs/11702511447/job/32590842412
edit:
I completely misread your question, sorry about that.
It would actually speed up the postinstall process, we may want it. I will add it in another PR, I keep it as it currently is in this PR
Proposed change
Setup Nx Cloud Id for PAT support
See https://nx.dev/ci/recipes/security/personal-access-tokens for details
Related issues
- No issue associated -