Skip to content

Conversation

@avivkeller
Copy link
Member

@avivkeller avivkeller commented Jan 8, 2026

Modernize the CI with improvements for brevity and speed

  • CI / Lint
  • CI / Smoketests
  • CI / Test - macos-latest (Node.js 18.x)
  • CI / Test - macos-latest (Node.js 20.x)
  • CI / Test - macos-latest (Node.js 22.x)
  • CI / Test - macos-latest (Node.js 24.x)
  • CI / Test - ubuntu-latest (Node.js 18.x)
  • CI / Test - ubuntu-latest (Node.js 20.x)
  • CI / Test - ubuntu-latest (Node.js 22.x)
  • CI / Test - ubuntu-latest (Node.js 24.x)
  • CI / Test - windows-latest (Node.js 18.x)
  • CI / Test - windows-latest (Node.js 20.x)
  • CI / Test - windows-latest (Node.js 22.x)
  • CI / Test - windows-latest (Node.js 24.x)
  • Check PR title / Validate PR title

@avivkeller avivkeller marked this pull request as ready for review January 8, 2026 21:00
Copilot AI review requested due to automatic review settings January 8, 2026 21:00
@avivkeller avivkeller requested a review from a team as a code owner January 8, 2026 21:00
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR modernizes the CI infrastructure by replacing commitlint with GitHub Actions-based PR title validation and consolidating CI workflows. Key changes include:

  • Removal of commitlint dependencies and local commit message validation
  • Migration from nodejs.yml to a simplified ci.yml workflow
  • Addition of automated PR title validation using GitHub Actions

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
package.json Removes @commitlint/cli and @commitlint/config-conventional dependencies
package-lock.json Removes commitlint and related dependency entries; also removes "peer" flags from multiple packages
commitlint.config.js Deletes commitlint configuration file
.husky/commit-msg Removes husky hook for commit message validation
.github/workflows/validate-pr-title.yml Adds new workflow for PR title validation using amannn/action-semantic-pull-request
.github/workflows/nodejs.yml Removes old CI workflow
.github/workflows/ci.yml Creates new consolidated CI workflow with lint and test jobs
.github/workflows/dependency-review.yml Updates workflow name and step descriptions
.github/dependabot.yml Changes GitHub Actions update schedule from daily to monthly and adds grouping
.cspell.json Adds "amannn" and removes "commitlint" from dictionary

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.72%. Comparing base (236e9e8) to head (92de822).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4620   +/-   ##
=======================================
  Coverage   89.72%   89.72%           
=======================================
  Files           9        9           
  Lines        1401     1401           
  Branches      448      448           
=======================================
  Hits         1257     1257           
  Misses        144      144           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 236e9e8...92de822. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@avivkeller avivkeller marked this pull request as draft January 8, 2026 22:15
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.

1 participant