Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 09e4f4c

Browse filesBrowse files
nsinfoPROaduh95
authored andcommitted
tools: use LTS Node.js in notify-on-push workflow
Without pinning Node.js, the runner defaults to Node 20 and npx falls back to core-validate-commit@5.0.1 instead of 6.0.0 (requires Node 22+). Version 5 does not recognise the ffi subsystem, causing false invalid-commit alerts. Signed-off-by: Nenad Spasenic <40522817+nsinfoPRO@users.noreply.github.com> PR-URL: #63084 Fixes: #63070 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com>
1 parent 8b91526 commit 09e4f4c
Copy full SHA for 09e4f4c

1 file changed

+7Lines changed: 7 additions & 0 deletions

File tree

Expand file treeCollapse file tree
Open diff view settings
Filter options
Expand file treeCollapse file tree
Open diff view settings
Collapse file

‎.github/workflows/notify-on-push.yml‎

Copy file name to clipboardExpand all lines: .github/workflows/notify-on-push.yml
+7Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,9 @@ name: Notify on Push
77
permissions:
88
contents: read
99

10+
env:
11+
NODE_VERSION: lts/*
12+
1013
jobs:
1114
notifyOnForcePush:
1215
name: Notify on Force Push on `main`
@@ -33,6 +36,10 @@ jobs:
3336
# cannot use ubuntu-slim here because rtCamp/action-slack-notify is dockerized
3437
runs-on: ubuntu-24.04-arm
3538
steps:
39+
- name: Install Node.js
40+
uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6.4.0
41+
with:
42+
node-version: ${{ env.NODE_VERSION }}
3643
- name: Validate commits
3744
run: echo "$COMMITS" | npx -q core-validate-commit -
3845
id: commit-check

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.