Skip to content

Commit

Permalink
ci: add install step in testing that in the end skips cache (#1339)
Browse files Browse the repository at this point in the history
  • Loading branch information
derberg authored Apr 4, 2024
1 parent 1d959af commit 5895a8a
Show file tree
Hide file tree
Showing 2 changed files with 27 additions and 5 deletions.
9 changes: 7 additions & 2 deletions .github/workflows/if-nodejs-pr-testing.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
git config --global core.eol lf
- if: steps.should_run.outputs.shouldrun == 'true'
name: Checkout repository
uses: actions/checkout@v3
uses: actions/checkout@v4
- if: steps.should_run.outputs.shouldrun == 'true'
name: Check if Node.js project and has package.json
id: packagejson
Expand All @@ -53,7 +53,7 @@ jobs:
id: lockversion
- if: steps.packagejson.outputs.exists == 'true'
name: Setup Node.js
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: "${{ steps.lockversion.outputs.version }}"
cache: 'npm'
Expand All @@ -65,9 +65,14 @@ jobs:
continue-on-error: true
- if: steps.first-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Clear NPM cache and install deps again
id: second-installation
run: |
npm cache clean --force
npm ci
continue-on-error: true
- if: steps.second-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Install without cache
run: npm ci --no-cache
- if: steps.packagejson.outputs.exists == 'true'
name: Test
run: npm test --if-present
Expand Down
23 changes: 20 additions & 3 deletions .github/workflows/if-nodejs-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ jobs:
git config --global core.autocrlf false
git config --global core.eol lf
- name: Checkout repository
uses: actions/checkout@v3
uses: actions/checkout@v4
- name: Check if Node.js project and has package.json
id: packagejson
run: test -e ./package.json && echo "exists=true" >> $GITHUB_OUTPUT || echo "exists=false" >> $GITHUB_OUTPUT
Expand All @@ -50,14 +50,26 @@ jobs:
id: lockversion
- if: steps.packagejson.outputs.exists == 'true'
name: Setup Node.js
uses: actions/setup-node@v3
uses: actions/setup-node@v4
with:
node-version: "${{ steps.lockversion.outputs.version }}"
cache: 'npm'
cache-dependency-path: '**/package-lock.json'
- if: steps.packagejson.outputs.exists == 'true'
name: Install dependencies
run: npm install
id: first-installation
run: npm ci
continue-on-error: true
- if: steps.first-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Clear NPM cache and install deps again
id: second-installation
run: |
npm cache clean --force
npm ci
continue-on-error: true
- if: steps.second-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Install without cache
run: npm ci --no-cache
- if: steps.packagejson.outputs.exists == 'true'
name: Run test
run: npm test --if-present
Expand Down Expand Up @@ -104,9 +116,14 @@ jobs:
continue-on-error: true
- if: steps.first-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Clear NPM cache and install deps again
id: second-installation
run: |
npm cache clean --force
npm ci
continue-on-error: true
- if: steps.second-installation.outcome != 'success' && steps.packagejson.outputs.exists == 'true'
name: Install without cache
run: npm ci --no-cache
- if: steps.packagejson.outputs.exists == 'true'
name: Add plugin for conventional commits for semantic-release
run: npm install --save-dev [email protected]
Expand Down

0 comments on commit 5895a8a

Please sign in to comment.