Repository navigation
docs(preact-query/utils): pin the preact and React 'useSyncExternalStore' source links to the copied commits - #12288
Conversation
…ore' source links to the copied commits
|
|
View your CI Pipeline Execution ↗ for commit 4d2ab36
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
size-limit report 📦
|
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configuration
📒 Files selected for processing: 1
✨ Finishing Touches
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 |
🎯 Changes
Pin the source links in the
useSyncExternalStorecomment ofpackages/preact-query/src/utils.tsto commit permalinks, since themainlinks no longer point at the referenced code:compat/src/hooks.js#L8-L54now points to9903119ede99e949d4c146eb42935ee0008caf7f, the revision whoseuseSyncExternalStoreanddidSnapshotChangematch the copied code. Onmain, the implementation has since moved tocompat/src/render.js.useSyncExternalStoreShimClient.js#L84now points to9cdf8a99edcfd94d7420835ea663edca04237527, the last revision where line 84 isconst [{inst}, forceUpdate] = useState(...). Onmain, that line has shifted.✅ Checklist
pnpm run test:pr, or these tests do not apply to this pull request.🚀 Release Impact