upgrade: idle package upgrade for Solid 2.0#888
Conversation
🦋 Changeset detectedLatest commit: f8eb78d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
src/createIdleTimer.ts`
isServerimport moved to@solidjs/webonMount→onSettledbatch()calls removed (replaced with sequential signal writes)INTERNAL_OPTIONS(ownedWrite: true)noopfrom utilstest/index.test.tsflushimport fromsolid-jsflush()after every signal-writing operation: afteronSettledfires (via the existing call), after eachvi.advanceTimersByTime()that triggers timer callbacks, aftertimer.start(),timer.reset(),timer.stop(),timer.triggerIdle(), and after DOM event dispatches that invokehandleEventtest/server.test.ts— new SSR safety testREADME.md— fixed default events list (removed stale"mousewheel", only"wheel"is in the actual defaults).changeset/idle-solid2-migration.md— major version bump changesetSummary by CodeRabbit
Bug Fixes
Tests
Updates