Skip to content

fix(ci): align workflow Node.js version with package engines - #520

Merged
gschier merged 2 commits into
mountain-loop:mainfrom
MGPOCKY:fix/ci-node-version-mismatch
Aug 2, 2026
Merged

fix(ci): align workflow Node.js version with package engines#520
gschier merged 2 commits into
mountain-loop:mainfrom
MGPOCKY:fix/ci-node-version-mismatch

Conversation

@MGPOCKY

@MGPOCKY MGPOCKY commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Why

Package metadata pins Node via .node-version as 24, but the Flathub workflow still uses Node 22.

What changed

  • .github/workflows/flathub.yml
    • node-version: "22"node-version: "24"

Release workflows that intentionally track lts/* are left unchanged.

Test plan

  • .node-version is still 24
  • Flathub workflow now uses Node 24
  • Relevant CI jobs on this branch look healthy

Found while auditing CI/package version consistency across popular repos.

@MGPOCKY
MGPOCKY marked this pull request as ready for review July 31, 2026 10:41
@github-actions github-actions Bot added the contribution: missing template Community PR is missing enough of the pull request template to review. label Jul 31, 2026
@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown

Thanks for the PR. This appears to match Yaak's contribution policy and is awaiting review by @gschier.

This only means the PR is in scope for review. It does not mean the change has been reviewed or accepted for merge.

@gschier gschier added contribution: explicit permission Community PR links feedback where @gschier explicitly allowed the work. and removed contribution: missing template Community PR is missing enough of the pull request template to review. labels Aug 2, 2026
@github-actions
github-actions Bot requested a review from gschier August 2, 2026 15:06
@gschier
gschier merged commit 784a3d3 into mountain-loop:main Aug 2, 2026
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution: explicit permission Community PR links feedback where @gschier explicitly allowed the work.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants