Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 2, 2026

fix: #74

Consolidates all CI jobs to Node.js 20 only, upgrades all GitHub Actions to their latest stable versions, and restricts CI workflow triggers to run only on master branch pushes and pull requests.

Changes

Node.js Version Updates

  • lint job: Added explicit Node.js 20 version specification
  • test job: Removed Node.js 18 from matrix, now runs only on Node.js 20

All jobs (lint, coverage, test, semantic-release) now consistently use Node.js 20.

GitHub Actions Version Updates

  • actions/checkout: Upgraded from v2 to v4 (all 4 instances)
  • actions/setup-node: Upgraded from v2.1.5 and v3 to v4 (all 4 instances)
  • coverallsapp/github-action: Upgraded from master to v2
  • mirromutth/mysql-action: Kept at v1.1 (already at latest stable version)

CI Trigger Configuration

  • Workflow triggers: Changed from on: [push, pull_request] to only run on:
    • Pushes to master branch
    • All pull requests

This prevents unnecessary CI runs on pushes to feature branches, reducing CI resource usage.

@hsluoyz
Copy link
Member

hsluoyz commented Feb 2, 2026

@nodece @Shivansh-yadav13 please review

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@hsluoyz hsluoyz marked this pull request as ready for review February 2, 2026 18:08
Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copilot AI changed the title [WIP] Upgrade CI scripts to run with Node.js 20 Upgrade CI to Node.js 20, drop Node.js 18 Feb 2, 2026
Copilot AI requested a review from hsluoyz February 2, 2026 18:11
@hsluoyz
Copy link
Member

hsluoyz commented Feb 2, 2026

@copilot also need to upgrade the used actions' versions in the CI scripts

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 2, 2026

@copilot also need to upgrade the used actions' versions in the CI scripts

Done! Upgraded all GitHub Actions to latest stable versions in commit e1fe4aa:

  • actions/checkout: v2 → v4
  • actions/setup-node: v2.1.5/v3 → v4
  • coverallsapp/github-action: master → v2

Copilot AI changed the title Upgrade CI to Node.js 20, drop Node.js 18 Upgrade CI to Node.js 20 and update GitHub Actions versions Feb 2, 2026
@hsluoyz
Copy link
Member

hsluoyz commented Feb 2, 2026

@copilot - also need to only run CI for only pull requests and pushes to the master branch (so push to other branches won't run CI)

on:
push:
branches:
- master
pull_request:

Co-authored-by: hsluoyz <3787410+hsluoyz@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Feb 2, 2026

@copilot - also need to only run CI for only pull requests and pushes to the master branch (so push to other branches won't run CI)

on:
...

Done! Updated CI trigger configuration in commit 569a1ae. The workflow now only runs on:

  • Pushes to master branch
  • All pull requests

Copilot AI changed the title Upgrade CI to Node.js 20 and update GitHub Actions versions Upgrade CI to Node.js 20, update GitHub Actions versions, and restrict CI triggers Feb 2, 2026
@hsluoyz hsluoyz changed the title Upgrade CI to Node.js 20, update GitHub Actions versions, and restrict CI triggers feat: upgrade CI to Node.js 20, update GitHub Actions versions, and restrict CI triggers Feb 2, 2026
@hsluoyz hsluoyz merged commit d585f27 into master Feb 2, 2026
4 of 6 checks passed
hsluoyz added a commit that referenced this pull request Feb 2, 2026
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.

[feature] upgrade CI script's versions

3 participants