-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
chore: run CI tests on Node 20 #9706
Conversation
|
uh oh. I've run the tests three times now and they've failed in the same way each time:
The Svelte, Vite, and |
In make_trackable, we're using tracked[property] to access searchParams: kit/packages/kit/src/utils/url.js Line 108 in 8a950aa
Its impl in Node 20 is if (this.#searchParams == null) {
this.#searchParams = new URLSearchParams(this.search);
setURLSearchParamsContext(this.#searchParams, this);
}
return this.#searchParams; so when it tries to access this.search, it triggers the tracking callback because the search property was patched in the previous iteration. I think at some point we used proxies instead for this, I'll try to dig up the history, perhaps we can switch to that depending on the reason or if I'm just misremembering. or alternatively hack around it by accessing searchParams before we patching search. EDIT: There's probably a simpler solution, running tests for it now, will push if they pass. |
Test are passing now with 20.1.0. |
amazing! thank you so much @gtm-nayan ! |
I forgot to add a changeset here after this added changes to sveltekit, so did that here: 6e2efcf |
No description provided.