Skip to content
This repository has been archived by the owner on Sep 29, 2023. It is now read-only.

Commit

Permalink
Update Electron to 21 + Node to 16 (#1550)
Browse files Browse the repository at this point in the history
  • Loading branch information
TheCleric committed Aug 3, 2023
1 parent fae8edd commit 051622d
Show file tree
Hide file tree
Showing 13 changed files with 9,691 additions and 1,198 deletions.
12 changes: 6 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@ jobs:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
- name: Use Node.js 19
- name: Use Node.js 20
uses: actions/setup-node@v2
with:
node-version: 19
node-version: 20
cache: 'npm'
cache-dependency-path: |
npm-shrinkwrap.json
Expand All @@ -37,10 +37,10 @@ jobs:
runs-on: windows-latest # Doesn't work on headless ubuntu, and is slow on mac
steps:
- uses: actions/checkout@v2
- name: Use Node.js 19
- name: Use Node.js 20
uses: actions/setup-node@v2
with:
node-version: 19
node-version: 20
cache: 'npm'
cache-dependency-path: |
npm-shrinkwrap.json
Expand All @@ -59,8 +59,8 @@ jobs:
strategy:
matrix:
node-version:
- '19'
- '12' # the oldest we require in package.json -> engines.node, to check we run on this minimum
- '20'
- '16' # the oldest we require in package.json -> engines.node, to check we run on this minimum
platform: [ubuntu-latest, macos-latest, windows-latest]
runs-on: ${{ matrix.platform }}
steps:
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1 # Setup .npmrc file to publish to npm
with:
node-version: '19' # Align the version of Node here with ci.yml.
node-version: '20' # Align the version of Node here with ci.yml.
registry-url: 'https://registry.npmjs.org'
- run: npm ci --no-fund # Will also (via `prepare` hook): 1. install ./app, 2. build
- run: npm run test:playwright
Expand All @@ -23,7 +23,7 @@ jobs:
- uses: actions/checkout@v2
- uses: actions/setup-node@v1 # Setup .npmrc file to publish to npm
with:
node-version: '19' # Align the version of Node here with ci.yml.
node-version: '20' # Align the version of Node here with ci.yml.
registry-url: 'https://registry.npmjs.org'
- run: npm ci --no-fund # Will also (via `prepare` hook): 1. install ./app, 2. build
- run: npm run test:noplaywright
Expand Down
6 changes: 4 additions & 2 deletions HACKING.md
Original file line number Diff line number Diff line change
Expand Up @@ -127,8 +127,10 @@ When a new major [Electron release](https://github.com/electron/electron/release
(also via the [Releases page](https://github.com/electron/electron/releases) and [the blog](https://www.electronjs.org/blog/), the content is different),
grepping our codebase for every changed API.
- If called for by the breaking changes, perform the necessary API changes
3. Bump `src/constants.ts` / `DEFAULT_ELECTRON_VERSION` & `DEFAULT_CHROME_VERSION`
and `app / package.json / devDeps / electron`
3. Bump
- `src/constants.ts` / `DEFAULT_ELECTRON_VERSION` & `DEFAULT_CHROME_VERSION`
- `package.json / devDeps / electron`
- `app / package.json / devDeps / electron`
4. On Windows, macOS, Linux, test for regression and crashes:
1. With `npm test` and `npm run test:manual`
2. With extra manual testing
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ Whatsapp Web ([HN thread](https://news.ycombinator.com/item?id=10930718)). Nativ

Install Nativefier globally with `npm install -g nativefier` . Requirements:

- macOS 10.10+ / Windows / Linux
- [Node.js](https://nodejs.org/)12.9 and npm ≥ 6.9
- macOS 10.13+ / Windows / Linux
- [Node.js](https://nodejs.org/)16.9 and npm ≥ 7.10

Optional dependencies:

Expand Down

0 comments on commit 051622d

Please sign in to comment.