Repository navigation
Conversation
✅ Deploy Preview for reactlynx-use ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request transitions the @lynx-js/react-use package to be ESM-only by removing CommonJS support, updating package.json entry points, removing the CJS build configuration from rslib.config.ts, and updating the tree-shaking tests. The reviewer correctly points out that because removing CommonJS support is a breaking change, the changeset should specify a major version bump instead of a minor bump to comply with Semantic Versioning.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| @@ -0,0 +1,5 @@ | |||
| --- | |||
| "@lynx-js/react-use": minor | |||
There was a problem hiding this comment.
Since this pull request removes CommonJS support and is explicitly marked as a breaking change (feat!:), the changeset should specify a major version bump instead of minor to comply with Semantic Versioning (SemVer).
| "@lynx-js/react-use": minor | |
| "@lynx-js/react-use": major |
03a9fd4 to
6e4c778
Compare
f0rdream
changed the title
|
Closing for now since the Publint message is non-blocking and the current package remains publishable. |
1 participant
Publish @lynx-js/react-use as a bundleless ESM-only package for Rspeedy/Rspack consumers. Remove the CommonJS build and export, point the package entry to the ESM artifact, and make the tree-shaking coverage resolve through the public package exports. This also removes the dual-package declaration warning.