Skip to content

Commit

Permalink
various changes to CI pipeline
Browse files Browse the repository at this point in the history
- use only node versions under maintenance
- use faster buildjet runners
- do not test on lint job (seemed redundant)
- bump checkout and setup-node actions
  • Loading branch information
vroldanbet committed Mar 12, 2024
1 parent 6d4c459 commit 9ee1598
Showing 1 changed file with 13 additions and 16 deletions.
29 changes: 13 additions & 16 deletions .github/workflows/authzed-node.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,17 @@ on:
types: [published]
jobs:
test:
name: Lint and Test
runs-on: ubuntu-latest
name: Lint
runs-on: "buildjet-2vcpu-ubuntu-2204"
strategy:
matrix:
node-version: [16, 17, 18]
node-version: [18, 20, 21]
steps:
- uses: actions/checkout@v2
- uses: actions/checkout@v4
- uses: "authzed/action-spicedb@v1"
with:
version: "latest"
- uses: actions/setup-node@v1
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
- uses: bahmutov/npm-install@v1
Expand All @@ -38,20 +38,17 @@ jobs:
- name: Run Yarn lint
run: CI=true yarn lint
working-directory: ./
- name: Run Yarn tests
run: CI=true yarn only-run-tests
working-directory: ./
publish-npm:
name: Publish to NPM
needs:
- test
runs-on: ubuntu-latest
if: github.event_name == 'release'
steps:
- uses: actions/checkout@v1
- uses: actions/setup-node@v1
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 16
node-version: 18
- run: npm install
- run: npm test
- uses: battila7/get-version-action@v2
Expand All @@ -62,16 +59,16 @@ jobs:
access: public
build-js-client:
name: Build and Test JS client
runs-on: ubuntu-latest
runs-on: "buildjet-2vcpu-ubuntu-2204"
strategy:
matrix:
node-version: [16, 17, 18]
node-version: [18, 20, 21]
steps:
- uses: actions/checkout@v2
- uses: "authzed/action-spicedb@v1"
with:
version: "latest"
- uses: actions/setup-node@v1
- uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
- run: npm install
Expand Down Expand Up @@ -103,9 +100,9 @@ jobs:
uses: actions/download-artifact@v2
with:
name: js-client
- uses: actions/setup-node@v2
- uses: actions/setup-node@v4
with:
node-version: 16
node-version: 18
- run: npm install
- uses: battila7/get-version-action@v2
- uses: JS-DevTools/npm-publish@v1
Expand Down

0 comments on commit 9ee1598

Please sign in to comment.