Skip to content

Commit 31357fd

Browse files
Copilotroblarsen
andauthored
ci: drop Node 20 from matrix, migrate publish to Node 22 (#3414)
* Initial plan * ci: drop Node 20 from test matrix, use Node 22 for publish, update docs Agent-Logs-Url: https://github.com/h5bp/html5-boilerplate/sessions/fc3e7fbd-ad7f-461e-adc2-a467bd7c365b Co-authored-by: roblarsen <[email protected]> --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: roblarsen <[email protected]>
1 parent 3d3ea0f commit 31357fd

3 files changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
- name: Setup Node
4949
uses: actions/setup-node@2028fbc5c25fe9cf00d9f06a71cc4710d4507903 # v6.0.0
5050
with:
51-
node-version: 20
51+
node-version: 22
5252
registry-url: https://registry.npmjs.org/
5353
- name: publish npm
5454
run: npm publish

.github/workflows/test.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ jobs:
1616
strategy:
1717
matrix:
1818
node-version:
19-
- 20
2019
- 22
2120
- 24
2221
os: [ubuntu-latest, windows-latest]

docs/about-this-repo.md

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,8 @@ purposes.
4848
Our branch protection rules are as follows:
4949

5050
- We require a pull request (PR) with one approving reviewer to merge code
51-
- In addition to the PR and approving reviewer, we require three status checks
51+
- In addition to the PR and approving reviewer, we require two status checks
5252
to pass before code can be merged
53-
_ Build with Node 20
5453
_ Build with Node 22
5554
_ Build with Node 24
5655
- We _allow_ force pushes for project admins. While force pushes can create
@@ -62,7 +61,7 @@ Our branch protection rules are as follows:
6261

6362
- We run a simple _build status_ check. This is the most basic test you can run
6463
and is absolutely vital. If you can't build your project you're in trouble.
65-
Currently we're testing against Node 16 and 18.
64+
Currently we're testing against Node 22 and 24.
6665
- We take advantage of our access to _CodeQL analysis_ Free for research and
6766
open source don't you know :) We don't have a ton of surface area to cover,
6867
but it's nice to have this powerful code scanning tool available to us.

0 commit comments

Comments
 (0)