Skip to content

build(*): removing node 22 and adding node 26 - #1791

Merged
dkalinovInfra merged 2 commits into
masterfrom
nodejs-build-versions
Sep 2, 2026
Merged

build(*): removing node 22 and adding node 26#1791
dkalinovInfra merged 2 commits into
masterfrom
nodejs-build-versions

Conversation

@kdinev

@kdinev kdinev commented Sep 1, 2026

Copy link
Copy Markdown
Member

Description

Related Issue

Closes #

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Refactoring / code cleanup
  • Build / CI configuration change

Affected Packages

  • igniteui-cli (packages/cli)
  • @igniteui/cli-core (packages/core)
  • @igniteui/angular-templates (packages/igx-templates)
  • @igniteui/angular-schematics (packages/ng-schematics)
  • @igniteui/mcp-server (packages/igniteui-mcp)

Checklist

  • I have tested my changes locally (npm run test)
  • I have built the project successfully (npm run build)
  • I have run the linter (npm run lint)
  • I have added/updated tests as needed
  • My changes do not introduce new warnings or errors

Additional Context

Copilot AI 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.

🟢 Approval recommended

The workflow change is small, scoped to CI configuration, and appears internally consistent (matrix + conditional coverage/publish logic).

Pull request overview

Updates the GitHub Actions CI Node.js version matrix to stop testing on Node 22 and start testing on Node 26, keeping Node 24 in the matrix, and slightly refines the Coveralls action comment formatting.

Changes:

  • CI matrix: replaced 22.x with 26.x (now tests on 26.x and 24.x).
  • Kept coverage + Coveralls publishing restricted to the 24.x job, while moving the Coveralls action version note inline.
File summaries
File Description
.github/workflows/nodejs.yml Updates Node.js test matrix from 22/24 to 26/24 and tweaks the Coveralls uses: comment formatting.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 87.816%. remained the same — nodejs-build-versions into master

@dkalinovInfra
dkalinovInfra merged commit 98fa225 into master Sep 2, 2026
4 checks passed
@dkalinovInfra
dkalinovInfra deleted the nodejs-build-versions branch September 2, 2026 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants