Skip to content

Commit

Permalink
fix(package-npm): build on Ubuntu 20.04 by default
Browse files Browse the repository at this point in the history
  • Loading branch information
segevfiner authored Aug 1, 2024
1 parent 66df2de commit 9bfd32c
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 3 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/package-npm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,10 @@ on:
description: The Emscripten version
default: ${{vars.EMSCRIPTEN_VERSION || '3.1.55'}}
type: string
ubuntu-version:
description: The version of the Ubuntu runner image
default: ${{vars.UBUNTU_VERSION || '20.04'}}
type: string
secrets:
NODE_AUTH_TOKEN:
description: An authentication token for npm
Expand All @@ -27,7 +31,7 @@ on:
jobs:
build_wasm:
name: Build Wasm binaries
runs-on: ubuntu-latest
runs-on: ubuntu-${{inputs.ubuntu-version}}
continue-on-error: true
steps:
- name: Checkout repository
Expand Down Expand Up @@ -61,7 +65,7 @@ jobs:
runs-on: ${{matrix.os}}
strategy:
matrix:
os: [ubuntu-latest, windows-latest, macos-latest]
os: [ubuntu-${{inputs.ubuntu-version}}, windows-latest, macos-latest]
steps:
- name: Checkout repository
uses: actions/checkout@v4
Expand All @@ -87,7 +91,7 @@ jobs:
package:
name: Publish NodeJS package
needs: [build_wasm, build_node]
runs-on: ubuntu-latest
runs-on: ubuntu-${{inputs.ubuntu-version}}
environment:
name: ${{inputs.environment-name}}
url: https://www.npmjs.com/package/${{inputs.package-name}}
Expand Down
4 changes: 4 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,10 @@ inputs:
description: The Emscripten version
default: ${{vars.EMSCRIPTEN_VERSION || '3.1.55'}}
type: string
ubuntu-version:
description: The version of the Ubuntu runner image
default: ${{vars.UBUNTU_VERSION || '20.04'}}
type: string
secrets:
NODE_AUTH_TOKEN:
description: An authentication token for npm
Expand Down

0 comments on commit 9bfd32c

Please sign in to comment.