Skip to content

Commit

Permalink
Speed up cicd, no point running two containers when one can do all fa…
Browse files Browse the repository at this point in the history
…ster
  • Loading branch information
duttonw committed Apr 1, 2024
1 parent d36cb47 commit c842b6c
Show file tree
Hide file tree
Showing 2 changed files with 3,760 additions and 1,745 deletions.
75 changes: 9 additions & 66 deletions .github/workflows/compile.js.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,77 +12,12 @@ concurrency:
cancel-in-progress: true

jobs:
validate:

runs-on: ubuntu-latest
strategy:
matrix:
node: [ '20']
name: Lint test on Node ${{ matrix.node }}
steps:
- run: echo ${{github.ref}}

- uses: actions/checkout@v4

- name: Use Node.js
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
cache: 'npm'
#always-auth: 'true'
#registry-url: 'https://nexus.tools.services.qld.gov.au/nexus/repository/npm_all/'

- name: Cache node modules
id: cache-npm
uses: actions/cache@v3
env:
cache-name: cache-node-modules
with:
# npm cache files are stored in `~/.npm` on Linux/macOS
path: ~/.npm
key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/package-lock.json') }}
restore-keys: |
${{ runner.os }}-build-${{ env.cache-name }}-
${{ runner.os }}-build-
${{ runner.os }}-
- if: ${{ steps.cache-npm.outputs.cache-hit != 'true' }}
name: List the state of node modules
continue-on-error: true
run: npm list

- name: npmrc #run on lint step (Which is cached)
run: |
npm -v
node -v
# cat /home/runner/work/_temp/.npmrc #only nice to test when registry is altered
#env:
# NODE_AUTH_TOKEN: ${{ secrets.NEXUSREADONLY2NPMTOKEN }}

- name: Install
run: | # Install packages
npm install --prefer-offline --no-audit --ignore-scripts --force
#env:
# NODE_AUTH_TOKEN: ${{ secrets.NEXUSREADONLY2NPMTOKEN }}

# `npm rebuild` will run all those post-install scripts for us.
- name: rebuild and prepare
run: npm rebuild && npm run prepare --if-present

- name: Lint
run: |
npm run lint
- name: Test
run: |
npm run test
build:
needs: validate
runs-on: ubuntu-latest
strategy:
matrix:
node: [ '20' ]
name: Build and Deploy on Node ${{ matrix.node }}
name: Lint, Test, Build and Deploy on Node ${{ matrix.node }}
steps:
- uses: actions/[email protected]

Expand Down Expand Up @@ -123,6 +58,14 @@ jobs:
- name: rebuild and prepare
run: npm rebuild && npm run prepare --if-present

- name: Lint
run: |
npm run lint
- name: Test
run: |
npm run test
- name: Build 🔧
run: | # build the files
npm run build
Expand Down
Loading

0 comments on commit c842b6c

Please sign in to comment.