-
-
Notifications
You must be signed in to change notification settings - Fork 4
Dev #98
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
base: main
Are you sure you want to change the base?
Dev #98
Changes from all commits
3a91eaf
76a4ce4
6d982d1
92ab360
80a83b0
a7dc760
b38d748
d6e6c72
0d5f483
d82c7d2
6e755f5
916eb1b
a78a490
520f5a6
b852f10
693b162
13e8921
9ecf55d
c753d50
ebd72ab
be2184d
9c5f224
1724630
478dfe3
2640d40
686d371
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -29,7 +29,7 @@ jobs: | |||||
| - uses: actions/setup-node@v4 | ||||||
| with: | ||||||
| node-version: '20' | ||||||
| - uses: pnpm/action-setup@v2.4.0 | ||||||
| - uses: pnpm/action-setup@v3.0.0 | ||||||
| name: Install pnpm | ||||||
| id: pnpm-install | ||||||
| with: | ||||||
|
|
@@ -45,14 +45,14 @@ jobs: | |||||
| run: | | ||||||
| echo "::set-output name=pnpm_cache_dir::$(pnpm store path)" | ||||||
|
|
||||||
| - uses: actions/cache@v3 | ||||||
| - uses: actions/cache@v4 | ||||||
| name: Setup pnpm cache | ||||||
| with: | ||||||
| path: ${{ steps.pnpm-cache.outputs.pnpm_cache_dir }} | ||||||
| key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||||||
| restore-keys: | | ||||||
| ${{ runner.os }}-pnpm-store- | ||||||
| - run: pnpm install --no-frozen-lockfile | ||||||
| - run: pnpm install | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🧩 Analysis chain🌐 Web query:
💡 Result:
To override this in CI, run:
CI detection is based on common CI env vars (via Sources: 🏁 Script executed: cat -n .github/workflows/nodejs.yml | sed -n '50,60p'Repository: tsparticles/angular Length of output: 568 🏁 Script executed: cat -n .github/workflows/nodejs.yml | sed -n '92,102p'Repository: tsparticles/angular Length of output: 563 🏁 Script executed: cat -n .github/workflows/nodejs.yml | sed -n '133,143p'Repository: tsparticles/angular Length of output: 464 🏁 Script executed: ls -la pnpm-lock.yaml 2>/dev/null && echo "Lockfile exists" || echo "Lockfile not found"Repository: tsparticles/angular Length of output: 144 🏁 Script executed: # Search for any commented-out lines with --no-frozen-lockfile in the workflow
rg "no-frozen-lockfile" .github/workflows/nodejs.ymlRepository: tsparticles/angular Length of output: 45 🏁 Script executed: # Check if there are any environment variables set for pnpm in the workflow
rg "PNPM_CONFIG" .github/workflows/nodejs.ymlRepository: tsparticles/angular Length of output: 45 Ensure In CI environments, pnpm auto-detects and defaults to Note: This also applies to lines 97 and 138 (commented agents job). 🤖 Prompt for AI Agents |
||||||
| #- run: npx nx-cloud start-ci-run | ||||||
| - run: pnpm run prettify:ci:readme | ||||||
| - run: pnpm run build:ci #--concurrency 3 | ||||||
|
|
@@ -71,7 +71,7 @@ jobs: | |||||
| - uses: actions/setup-node@v4 | ||||||
| with: | ||||||
| node-version: '20' | ||||||
| - uses: pnpm/action-setup@v2.4.0 | ||||||
| - uses: pnpm/action-setup@v3.0.0 | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. In the - version: 7
+ version: 8Committable suggestion
Suggested change
|
||||||
| name: Install pnpm | ||||||
| id: pnpm-install | ||||||
| with: | ||||||
|
|
@@ -87,14 +87,14 @@ jobs: | |||||
| run: | | ||||||
| echo "::set-output name=pnpm_cache_dir::$(pnpm store path)" | ||||||
|
||||||
|
|
||||||
| - uses: actions/cache@v3 | ||||||
| - uses: actions/cache@v4 | ||||||
| name: Setup pnpm cache | ||||||
| with: | ||||||
| path: ${{ steps.pnpm-cache.outputs.pnpm_cache_dir }} | ||||||
| key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||||||
| restore-keys: | | ||||||
| ${{ runner.os }}-pnpm-store- | ||||||
| - run: pnpm install --no-frozen-lockfile | ||||||
| - run: pnpm install | ||||||
| #- run: npx nx-cloud start-ci-run | ||||||
| - run: pnpm run prettify:ci:readme | ||||||
| - run: pnpm run build:ci #--concurrency 3 | ||||||
|
|
@@ -135,6 +135,6 @@ jobs: | |||||
| # key: ${{ runner.os }}-pnpm-store-${{ hashFiles('**/pnpm-lock.yaml') }} | ||||||
| # restore-keys: | | ||||||
| # ${{ runner.os }}-pnpm-store- | ||||||
| # - run: pnpm install --no-frozen-lockfile | ||||||
| # - run: pnpm install | ||||||
| # - name: Start Nx Agent ${{ matrix.agent }} | ||||||
| # run: npx nx-cloud start-agent | ||||||
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.
The set-output command is deprecated by GitHub Actions and will be removed. Replace 'echo "::set-output name=pnpm_cache_dir::$(pnpm store path)"' with 'echo "pnpm_cache_dir=$(pnpm store path)" >> $GITHUB_OUTPUT' to use the current recommended approach.