[2.4.0 stack 7/18] Public API: root exports, types, 100% coverage and type-aware lint gates - #512
[2.4.0 stack 7/18] Public API: root exports, types, 100% coverage and type-aware lint gates#512hyanmandian wants to merge 2 commits into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe barrel export now exposes additional helpers and public types. API tests verify the complete runtime and type surface, duplicate-free exports, and non-throwing behavior for invalid inputs. Type-aware linting and 100% coverage thresholds are enabled. ChangesPublic API expansion
Priority: ⬇️ Low Estimated code review effort: 3 (Moderate) | ~20 minutes Change: Feature Merge Risk: ⚪ Minimal · up to The public API expansion and its validation changes have no identified actionable merge risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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 |
Tree-shaking reportFails when a pre-existing export grows more than 20% and more than 256 B, or when importing every export that already existed on the base grows more than 5%. New exports never count as a regression. Pre-existing exports: 202766 B to 203016 B (+0.1%, gzip 67153 B). Full import on head: 628788 B (gzip 162664 B).
New exports
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## stack/05-new-utils #512 +/- ##
=======================================================
+ Coverage 98.75% 100.00% +1.24%
=======================================================
Files 152 152
Lines 2161 2161
Branches 664 664
=======================================================
+ Hits 2134 2161 +27
+ Misses 3 0 -3
+ Partials 24 0 -24
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
f4e1a60 to
e3c7bc0
Compare
e3c7bc0 to
827b077
Compare
827b077 to
7761122
Compare
53c80ee to
c6d1290
Compare
c6d1290 to
93a0a4b
Compare
93a0a4b to
b1a7a2b
Compare
b1a7a2b to
64c407f
Compare
|
@coderabbitai full review |
✅ Action performedFull review finished. |
Export State, StateCode, StateName, AddressInfo, CepProvider, BoletoInfo, GetBoletoInfoOptions, GenerateBoletoOptions, GenerateLicensePlateFormat, Holiday, HolidayType, PhoneMask, PhoneVersion, PhoneType, ParseCurrencyOptions, ParseCnpjOptions, IsValidCnpjOptions and every new util'"'"'s public types.
Turn on oxlint's type-aware rules with type checking and require 100% statement, branch, function and line coverage now that every source file meets both.
64c407f to
17a50c9
Compare
|
@coderabbitai full review |
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
What does this PR do?
Part 7 of 18 of the 2.4.0 release stack (main <- stack/01-tooling <- ... <- stack/11-testing). The package-root exports (
src/index.ts, 124 exports plus 10 deprecated aliases kept for compatibility), the public type re-exports and the Bun/Deno test runtime shim refresh. Closes with the commit that turns on type-aware lint with type checking and the 100% coverage thresholds, now that the whole tree meets them.Commits in this part (2)
How to review and merge
mainand merge it with a merge commit: the reviewed commits land unchanged and GitHub marks the seventeen parts below as merged.CHANGELOG.md(part 7); the build of every later part is compared export by export against the part below it.Stack