Skip to content

Commit

Permalink
Clearer Node.js compile GHA job names
Browse files Browse the repository at this point in the history
  • Loading branch information
emmercm committed Oct 13, 2023
1 parent 582423b commit dc58eed
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/node-compile.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,13 @@ jobs:
- path-filter
if: ${{ needs.path-filter.outputs.changes == 'true' || github.event_name != 'pull_request' }}
runs-on: ${{ matrix.os }}-latest
name: node-compile (${{ matrix.os }} ${{ matrix.arch }})
strategy:
matrix:
include:
# docker manifest inspect node:<version> | jq '.manifests[].platform'
- os: ubuntu
arch: amd64
build: |
NPM_PKG_VERSION=$(npm pkg get version | sed 's/"//g')
NODE_ARCH=amd64
Expand All @@ -63,6 +65,7 @@ jobs:
"./node_modules/.bin/caxa --input build/ --output igir -- \"{{caxa}}/node_modules/.bin/node\" \"{{caxa}}/index.js\" && ./igir --help"
tar cvf - igir | gzip --best > "${BIN_OUTPUT}"
- os: ubuntu
arch: arm/v7
build: |
NPM_PKG_VERSION=$(npm pkg get version | sed 's/"//g')
NODE_ARCH=arm/v7
Expand All @@ -74,6 +77,7 @@ jobs:
"./node_modules/.bin/caxa --input build/ --output igir -- \"{{caxa}}/node_modules/.bin/node\" \"{{caxa}}/index.js\" && ./igir --help"
tar cvf - igir | gzip --best > "${BIN_OUTPUT}"
- os: ubuntu
arch: arm64/v8
build: |
NPM_PKG_VERSION=$(npm pkg get version | sed 's/"//g')
NODE_ARCH=arm64/v8
Expand All @@ -86,6 +90,7 @@ jobs:
tar cvf - igir | gzip --best > "${BIN_OUTPUT}"
# TODO(cemmer): macOS arm64: https://github.com/github/roadmap/issues/528
- os: macos
arch: '*'
build: |
NPM_PKG_VERSION=$(npm pkg get version | sed 's/"//g')
NODE_ARCH=$(node --print 'process.arch')
Expand All @@ -96,6 +101,7 @@ jobs:
./igir --help
tar cvf - igir | gzip --best > "${BIN_OUTPUT}"
- os: windows
arch: '*'
build: |
$NPM_PKG_VERSION=$(npm pkg get version).replace('"','')
$NODE_ARCH=$(node --print 'process.arch')
Expand Down

0 comments on commit dc58eed

Please sign in to comment.