Skip to content

Stop testing on end-of-life versions of Node.js#1200

Merged
peterbarker merged 1 commit into
ArduPilot:masterfrom
cclauss:patch-12
Apr 14, 2026
Merged

Stop testing on end-of-life versions of Node.js#1200
peterbarker merged 1 commit into
ArduPilot:masterfrom
cclauss:patch-12

Conversation

@cclauss

@cclauss cclauss commented Apr 7, 2026

Copy link
Copy Markdown
Contributor

node-version: [latest, lts/*, lts/-1, lts/-2]
Currently:

tag Node.js version
latest v25.9.0
lts/* v24.14.1
lts/-1 v22.22.2
lts/-2 v20.20.2

https://nodejs.org/en/about/previous-releases
https://github.com/actions/setup-node?tab=readme-ov-file#supported-version-syntax

Note

Debian GNU/Linux 11 (bullseye) defaults to v12.22.12
Ubuntu 22.04.5 LTS (jammy) defaults to v12.22.9
Debian GNU/Linux 12 (bookworm) defaults to v18.20.4


% docker run -it ubuntu:22.04
# apt update && DEBIAN_FRONTEND=noninteractive apt install --yes nodejs && node --version && exit

Comment thread .github/workflows/test.yml Outdated
@peterbarker

peterbarker commented Apr 9, 2026 via email

Copy link
Copy Markdown
Contributor

`node-version: [latest, lts/*, lts/-1, lts/-2]`
 Currently:
* latest: v25.9.0
* lts/*: v24.14.1
* lts/-1: v22.22.2
* lts/-2: v20.20.2

* https://nodejs.org/en/about/previous-releases
* https://github.com/actions/setup-node?tab=readme-ov-file#supported-version-syntax
@cclauss

cclauss commented Apr 10, 2026

Copy link
Copy Markdown
Contributor Author

Yes. NVM LTS syntax is inexplicable. From the URL above...

NVM LTS syntax: lts/iron, lts/jod, lts/*, lts/-n

@cclauss cclauss requested a review from peterbarker April 10, 2026 00:20
@peterbarker

Copy link
Copy Markdown
Contributor

I'd still like @Williangalvani to say this is OK by him; I'm guessing BlueOS might care.

@Williangalvani Williangalvani left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@peterbarker peterbarker merged commit 83c4cc4 into ArduPilot:master Apr 14, 2026
22 checks passed
@peterbarker

Copy link
Copy Markdown
Contributor

Merged, thanks!

@cclauss cclauss deleted the patch-12 branch April 14, 2026 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants