Skip to content

Migrate project tooling to TypeScript 7#34

Merged
PabloZaiden merged 2 commits into
mainfrom
migrar-a-typescript-7-d59483f
Jul 8, 2026
Merged

Migrate project tooling to TypeScript 7#34
PabloZaiden merged 2 commits into
mainfrom
migrar-a-typescript-7-d59483f

Conversation

@PabloZaiden

Copy link
Copy Markdown
Owner

Summary

  • Updated project TypeScript tooling dependencies to TypeScript 7
  • Refreshed the Bun lockfile for the new dependency versions
  • Adjusted CI/release workflow configuration and README references for the migrated toolchain

Copilot AI review requested due to automatic review settings July 8, 2026 19:59

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

This PR migrates the project’s TypeScript tooling to TypeScript 7, adds a dedicated typecheck step to local/CI workflows, and updates documentation to reflect the new recommended development commands.

Changes:

  • Add a typecheck script (tsc --noEmit) and run it in PR CI and release prebuild.
  • Upgrade TypeScript tooling dependencies (notably @types/bun) and refresh bun.lock to TypeScript 7.0.2.
  • Update README development instructions to include the new typecheck step.

Reviewed changes

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

Show a summary per file
File Description
README.md Documents running bun run typecheck as part of the local development command sequence.
package.json Adds a typecheck script and introduces TypeScript 7 to devDependencies.
bun.lock Updates locked dependency graph for TypeScript 7 / updated Bun types.
.github/workflows/pr-tests.yml Runs bun run typecheck in PR CI before tests/build.
.github/workflows/binary-release.yml Runs bun run typecheck as part of the release prebuild checks.

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

Comment thread package.json Outdated
@PabloZaiden
PabloZaiden merged commit 21ef189 into main Jul 8, 2026
1 check passed
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