Repository navigation
feat(solid-pacer): support Solid 2 #307
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,4 @@ | ||
| { | ||
| "mode": "pre", | ||
| "tag": "rc" | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,11 @@ | ||
| --- | ||
| '@tanstack/solid-pacer': minor | ||
| --- | ||
|
|
||
| Support Solid 2. `@tanstack/solid-pacer` now requires `solid-js` 2 (`>=2.0.0-rc.19 <3.0.0`) and `@tanstack/solid-store` 0.12; stay on 0.24.x for Solid 1. | ||
|
|
||
| - The primitives keep their API. Reactive options are applied by split render effects (compute reads the merged options, the effect applies them), cleanup runs with `onCleanup` on the owner, and the `*Value` primitives read their source's initial value untracked. | ||
| - The `*Signal` primitives' signals allow writes from owned scopes, since timers and the `*Value` effects write them, and keep a function value as a plain value. | ||
| - `PacerProvider` renders the Solid 2 context without JSX, so the adapter depends on `solid-js` only. | ||
| - As with every Solid 2 signal, a write is read back after the next flush. When an option getter such as `enabled` reads a signal you just set, call `flush()` before calling the utility in the same handler. | ||
| - `@tanstack/solid-pacer-devtools` stays on Solid 1 until TanStack Devtools has a Solid 2 entry. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,12 +10,13 @@ | |
| "test:e2e": "PLAYWRIGHT_TEST_DIR=$PWD/tests/e2e playwright test --config ../../../playwright.config.ts" | ||
| }, | ||
| "dependencies": { | ||
| "@solidjs/web": "^2.0.0-rc.19", | ||
| "@tanstack/solid-pacer": "^0.24.0", | ||
| "solid-js": "^1.9.15" | ||
| "solid-js": "^2.0.0-rc.19" | ||
| }, | ||
| "devDependencies": { | ||
| "vite": "^8.3.2", | ||
| "vite-plugin-solid": "^2.11.14" | ||
| "@solidjs/vite-plugin": "^3.0.0-next.52", | ||
|
Comment on lines
+13
to
+18
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
|
||
| "vite": "^8.3.2" | ||
| }, | ||
| "browserslist": { | ||
| "production": [ | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Script executed:
Repository: TanStack/pacer
Length of output: 3597
Script executed:
Repository: TanStack/pacer
Length of output: 820
Scope the Solid devtools install instruction to Solid 1.
Line 44 says the devtools adapter does not run on Solid 2 apps. Line 46 still tells every Solid reader to install it. Limit line 46 to Solid 1. Also review the unconditioned Solid package entries at lines 54-55.
📝 Suggested fix