Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #23 +/- ##
=========================================
+ Coverage 0 92.19% +92.19%
=========================================
Files 0 7 +7
Lines 0 141 +141
Branches 0 40 +40
=========================================
+ Hits 0 130 +130
- Misses 0 11 +11 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
e17e4e8 to
cfa62cc
Compare
f1a8ef0 to
1b6a4e7
Compare
1b6a4e7 to
894193f
Compare
9d11ee9 to
d314e85
Compare
296f8b7 to
6a2edd0
Compare
6a2edd0 to
348272c
Compare
| "./lib/dev-pages-utils": { | ||
| "types": "./lib/dev-pages-utils/index.d.ts", | ||
| "default": "./lib/dev-pages-utils/index.js" | ||
| }, | ||
| "./dev-pages-utils": { | ||
| "types": "./lib/dev-pages-utils/index.d.ts", | ||
| "default": "./lib/dev-pages-utils/index.js" | ||
| }, |
There was a problem hiding this comment.
Updated package exports to support both ./dev-pages-utils and ./lib/dev-pages-utils path to support for typescript for <5.0 and bundlers like vite that need explicit path if it is specified. You can see here that the resolvePackageJsonExports is not available before 5.0.
/dev-pages-utils: For TypeScript 5.0+ which properly resolves package.json exports./lib/dev-pages-utils: For TypeScript <5.0 and bundlers (Vite) that need explicit paths
e13aa0a to
977902a
Compare
|
Closed since this it has been split into two the following PRs: feat: building the build tools into /lib and feat: Add dev pages utils permutation view |
Description
This PR enables direct consumption of the pre-built build-tools package from GitHub, eliminating the need for consumers to build it locally. A pre-push hook ensures the build artifacts are always up-to-date before publishing.
How is it tested
Backward compatibility: Tested by consuming this branch in an existing repository to verify no breaking changes
Follow-up PR demonstrates usage with the new PermutationsView component in chat-components Draft PR, and Components Draft PR
Build verification: Pre-push hook validates that build artifacts are current before each push
Related links, issue #, if available: n/a
How has this been tested?
Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness
CONTRIBUTING.md.CONTRIBUTING.md.Testing
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.