Repository navigation
Conversation
Contributor
There was a problem hiding this comment.
Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.
An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info
📝 Walkthrough
Merge Risk: ⚪ Minimal · up to
|
| Check name | Status | Explanation | Resolution |
|---|---|---|---|
| Docstring Coverage |  |
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 8 files. (95 skipped: 9… | Write docstrings for the functions missing them to satisfy the coverage threshold. |
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly summarizes the main change: replacing workspace compiler commands with the shared tsrs runner. Linked Issues check Check skipped because no linked issues were found for this pull request. Out of Scope Changes check Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage
-
Explanation
Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 8 files. (95 skipped: 95 unsupported.)
Resolution
Write docstrings for the functions missing them to satisfy the coverage threshold.
- Fix all pre-merge checks with AI
✨ Finishing Touches: 💡 2
-
📝 Generate docstrings: 💡
-
- Commit to this branch
- Create a new PR
🛠️ Fix failing CI checks: 💡
-
- Commit to this branch
- Create a new PR
🧪 Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.
Comment @coderabbitai help to get the list of available commands.
Deploying egg-v3 with
|
| Latest commit: |
1969853
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://3d12345c.egg-v3.pages.dev |
| Branch Preview URL: | https://chore-use-tsrs.egg-v3.pages.dev |
Deploying egg with
|
| Latest commit: |
1969853
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fba2a103.egg-cci.pages.dev |
| Branch Preview URL: | https://chore-use-tsrs.egg-cci.pages.dev |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #6076 +/- ##
==========================================
+ Coverage 83.65% 84.02% +0.37%
==========================================
Files 734 734
Lines 22639 22639
Branches 4560 4560
==========================================
+ Hits 18938 19022 +84
+ Misses 3201 3126 -75
+ Partials 500 491 -9 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
All 88 workspace
typecheckscripts now usescripts/tsrs.jsto reduce compiler time and keep compiler selection in one place. Repository compiler fixtures and the HTTP benchmark build use the same runner.The runner selects native
tsrson macOS ARM64 and Linux ARM64/x64. It uses@ts-rs/wasmon other platforms, including Windows.This removes compiler-only
typescriptdependencies and the@typescript/nativealias. TypeScript5.9remains for the SWC compiler API. Generated application templates keep their existing compiler dependencies.The median time for
ut run typecheckdecreased by 35%, a 1.54× speedup:typescript@7.0.2tsrs@0.15.0-ts7.1.0-dev.20260929Each compiler had one warmup and five interleaved measured runs across all 88 workspaces. The machine used an Apple M4 Max, macOS
26.2, and Node.js22.23.2.Before each run, the benchmark removed
*.tsbuildinfofiles; filesystem caches stayed warm. Both compilers used the same Node launcher;TSRS_BINARYselected the native TypeScript baseline. These compiler versions target different TypeScript revisions.Summary by CodeRabbit