Skip to content

Update to Node 24 - #22

Merged
carlansley merged 6 commits into
mainfrom
node24
Aug 27, 2026
Merged

Update to Node 24#22
carlansley merged 6 commits into
mainfrom
node24

Conversation

@carlansley

Copy link
Copy Markdown
Contributor

Fixes #21

@carlansley carlansley self-assigned this Aug 27, 2026
Copilot AI lite review requested due to automatic review settings August 27, 2026 17:42

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the package and CI/CD configuration to require Node.js 24+ and aligns release automation/docs for a new major version.

Changes:

  • Bumps package major version to 6.0.0 and sets engines.node to >=24.18.0.
  • Updates GitHub Actions workflows (CI, coverage reporting, publish) for newer Node/action versions and revised publishing flow.
  • Refreshes docs/legal headers (README/SECURITY/LICENSE/source headers) and minor code/test formatting.

Reviewed changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/timeout.ts Formatting-only change to RangeError throw style.
src/timeout.spec.ts Test formatting/renames and updated parallel test range construction.
src/options.ts Copyright year update.
src/index.ts Copyright year update.
src/error.ts Copyright year update.
SECURITY.md Corrects package name/links and updates supported major version range.
README.md Updates copyright line and example code formatting.
package.json Major version bump, Node engine bump to 24, script/tooling changes.
LICENSE.txt Copyright year update.
.github/workflows/publish.yml Consolidates release + PR beta publishing into one workflow.
.github/workflows/publish-beta.yml Removes the separate beta publish workflow.
.github/workflows/coverage.yml Splits base vs PR coverage runs and posts comparison report.
.github/workflows/codeql-analysis.yml Updates CodeQL action version.
.github/workflows/ci.yml Updates Node matrix to 24/26 and adds prepare step.
.github/workflows/check-published-scheduled.yml Workflow maintenance and env var adjustments.

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

Comment thread package.json
@carlansley
carlansley deployed to npm-publish August 27, 2026 17:57 — with GitHub Actions Active
@carlansley
carlansley deployed to npm-publish August 27, 2026 18:02 — with GitHub Actions Active
@carlansley
carlansley deployed to npm-publish August 27, 2026 18:37 — with GitHub Actions Active
@carlansley
carlansley deployed to npm-publish August 27, 2026 18:43 — with GitHub Actions Active
@github-actions

Copy link
Copy Markdown

Coverage after merging node24 into main will be

100.00% 0.00%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   error.ts100%100%100%100%
   index.ts100%100%100%100%
   timeout.spec.ts99.38%96%100%100%63
   timeout.ts100%100%100%100%

@github-actions

Copy link
Copy Markdown

Beta Published - Install Command: npm install @checkdigit/timeout@6.0.0-PR.22-e6a6

@carlansley
carlansley requested a review from ramaghanta August 27, 2026 19:20

@ramaghanta ramaghanta 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.

lftm

@carlansley
carlansley merged commit ff6a6a0 into main Aug 27, 2026
11 checks passed
@carlansley
carlansley deleted the node24 branch August 27, 2026 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update minimum Node version to 24

3 participants