Skip to content

chore: support TypeScript 7 and update compatibility docs. - #122

Merged
knightedcodemonkey merged 7 commits into
mainfrom
bananas
Jul 11, 2026
Merged

chore: support TypeScript 7 and update compatibility docs.#122
knightedcodemonkey merged 7 commits into
mainfrom
bananas

Conversation

@knightedcodemonkey

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 11, 2026 19:22
@codecov

codecov Bot commented Jul 11, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.75758% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 93.30%. Comparing base (6a5b7e1) to head (de54cbe).

Files with missing lines Patch % Lines
src/duel.js 75.75% 8 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #122      +/-   ##
==========================================
- Coverage   93.57%   93.30%   -0.27%     
==========================================
  Files           4        4              
  Lines        2209     2240      +31     
==========================================
+ Hits         2067     2090      +23     
- Misses        142      150       +8     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

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.

Pull request overview

Updates the project to target TypeScript 7 in development tooling, refreshes documented TypeScript compatibility requirements, and adjusts CI/publish workflows accordingly.

Changes:

  • Document TypeScript support range in README.md and keep peer range at >=5.5.0 <8.
  • Bump package version and dev tooling versions (notably typescript to ^7.0.2, tsx to ^4.23.0).
  • Update GitHub Actions workflows (Node patch versions, actions/checkout), and remove the dedicated TypeScript v7 compatibility workflow.

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
README.md Adds explicit TypeScript version requirement to the “Requirements” section.
package.json Bumps package version; updates TS/TSX dev deps; removes typescript-rc peer alias while keeping typescript-next.
package-lock.json Locks updated package/devDependency versions consistent with package.json.
.github/workflows/typescript-v7.yml Removes the standalone TypeScript v7/RC compatibility workflow.
.github/workflows/publish.yml Updates checkout action and Node patch version used for publishing.
.github/workflows/ci.yml Updates Node patch versions and checkout action used in CI.

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

Comment thread .github/workflows/ci.yml
Copilot AI review requested due to automatic review settings July 11, 2026 19:41

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.

Pull request overview

Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.

Copilot AI review requested due to automatic review settings July 11, 2026 19:57

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.

Pull request overview

Copilot reviewed 6 out of 7 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

.github/workflows/typescript-next.yml:27

  • After switching to typescript@next, the workflow no longer prints the resolved TypeScript version. This makes compatibility failures harder to diagnose from CI logs; consider restoring the version output (as the previous RC workflow did).
    .github/workflows/typescript-next.yml:37
  • This advisory job is intended to validate TypeScript-next compatibility, but it no longer runs the project's build script (the prior workflow did). Without a build step, CI may miss breakages that only surface when Duel runs end-to-end.

Copilot AI review requested due to automatic review settings July 11, 2026 20:05

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.

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Comment thread src/duel.js Outdated
Copilot AI review requested due to automatic review settings July 11, 2026 20:20

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.

Pull request overview

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Comment thread src/duel.js
Comment thread src/duel.js Outdated
Copilot AI review requested due to automatic review settings July 11, 2026 20:42

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.

Pull request overview

Copilot reviewed 11 out of 13 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • test/fixtures/mononpm/package-lock.json: Generated file

Comment thread test/monorepos.js Outdated
Comment thread src/duel.js
Copilot AI review requested due to automatic review settings July 11, 2026 20:49

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.

Pull request overview

Copilot reviewed 11 out of 13 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • test/fixtures/mononpm/package-lock.json: Generated file

@knightedcodemonkey
knightedcodemonkey merged commit 9c3a453 into main Jul 11, 2026
10 checks passed
@knightedcodemonkey
knightedcodemonkey deleted the bananas branch July 11, 2026 20:57
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.

2 participants