Skip to content

Commit

Permalink
Raise minimum Node version to 18
Browse files Browse the repository at this point in the history
  • Loading branch information
ranisalt committed Nov 4, 2023
1 parent 617ac4d commit ca7c541
Show file tree
Hide file tree
Showing 3 changed files with 15 additions and 15 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
test:
strategy:
matrix:
node-version: [16, 18, 20]
node-version: [18, 20]
os: [ubuntu-20.04, macos-11, macos-m1, windows-2019]

name: Test for node-${{ matrix.node-version }} on ${{ matrix.os }}
Expand Down Expand Up @@ -59,7 +59,7 @@ jobs:
name: Test on Alpine Linux
runs-on: ubuntu-latest
container:
image: node:16-alpine
image: node:18-alpine

steps:
- name: Install build deps
Expand All @@ -79,7 +79,7 @@ jobs:
test-freebsd:
strategy:
matrix:
node-version: [16, 18, 20]
node-version: [18, 20]

name: Test for node-${{ matrix.node-version }} on FreeBSD
runs-on: macos-12
Expand Down
18 changes: 9 additions & 9 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,11 @@ jobs:
with:
submodules: true

- name: Use Node.js 16
- name: Use Node.js 18
uses: actions/setup-node@v3
with:
cache: yarn
node-version: 16
node-version: 18
registry-url: https://registry.npmjs.org/

- name: Install dependencies
Expand All @@ -47,11 +47,11 @@ jobs:
with:
submodules: true

- name: Use Node.js 16
- name: Use Node.js 18
uses: actions/setup-node@v3
with:
cache: yarn
node-version: 16
node-version: 18

- name: Install dependencies
run: yarn install --frozen-lockfile
Expand All @@ -70,7 +70,7 @@ jobs:
name: Build on Alpine Linux
runs-on: ubuntu-latest
container:
image: node:16-alpine
image: node:18-alpine

steps:
- name: Install build deps
Expand Down Expand Up @@ -111,7 +111,7 @@ jobs:
with:
submodules: true

- uses: uraimo/run-on-arch-action@v2.2.0
- uses: uraimo/run-on-arch-action@v2
name: Package artifacts
id: build
with:
Expand All @@ -126,13 +126,13 @@ jobs:
ubuntu*|jessie|stretch|buster)
apt-get update -y
apt-get install -y curl
curl -fsSL https://deb.nodesource.com/setup_16.x | bash -
curl -fsSL https://deb.nodesource.com/setup_18.x | bash -
apt-get install -y make g++ python nodejs
npm install --global yarn
;;
alpine*)
apk add --update make g++ python3
apk add --no-cache --repository https://dl-cdn.alpinelinux.org/alpine/v3.15/main/ nodejs~=16 npm
apk add --no-cache --repository https://dl-cdn.alpinelinux.org/alpine/v3.17/main/ nodejs~=18 npm
npm install --global yarn
;;
esac
Expand Down Expand Up @@ -163,7 +163,7 @@ jobs:
with:
envs: 'npm_config_build_from_source'
prepare: |
pkg install -y gmake python3 yarn-node16
pkg install -y gmake python3 yarn-node18
run: |
yarn install --frozen-lockfile
yarn node-pre-gyp package
Expand Down
6 changes: 3 additions & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -68,7 +68,7 @@ The current prebuilt binaries are built and tested with the following systems:
- MacOS 11 (x86-64)
- MacOS 12 (ARM64 from v0.29.0)
- Windows Server 2019 (x86-64)
- Alpine Linux 3.13 (x86-64 from v0.28.1; ARM64 from v0.28.2)
- Alpine Linux 3.17+ (x86-64 from v0.28.1; ARM64 from v0.28.2)
- FreeBSD 13.1 (x86-64 from v0.29.1)

Binaries should also work for any version more recent than the ones listed
Expand All @@ -86,7 +86,7 @@ You **MUST** have a **node-gyp** global install before proceeding with the insta
along with GCC >= 5 / Clang >= 3.3. On Windows, you must compile under Visual
Studio 2015 or newer.

**node-argon2** works only and is tested against Node >=14.0.0.
**node-argon2** works only and is tested against Node >=18.0.0.

### OSX
To install GCC >= 5 on OSX, use [homebrew](http://brew.sh/):
Expand Down Expand Up @@ -149,7 +149,7 @@ you'll need to specify that in the command.
<summary>
I installed Node as a <a href="https://snapcraft.io/node">snap</a>, and I can't install node-argon2.
</summary>

This seems to be an issue related to snap (see [#345 (comment)](https://github.com/ranisalt/node-argon2/issues/345#issuecomment-1164178674)). Installing Node with another package manager, such as [asdf](https://asdf-vm.com/) or [nvm](https://github.com/nvm-sh/nvm), is a possible workaround.
</details>

Expand Down

0 comments on commit ca7c541

Please sign in to comment.