QwikDev / qwik

Instant-loading web apps, without effort
https://qwik.dev
MIT License
20.71k stars 1.29k forks source link

fea: switching tsm to ts-node in packagejson #6877

Open JerryWu1234 opened 2 weeks ago

JerryWu1234 commented 2 weeks ago

6856

What is it?

Description

Checklist:

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: 540042643542e68738e22470169a85599415fc0c

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 4 packages | Name | Type | | --------------------- | ----- | | create-qwik | Major | | @builder.io/qwik | Major | | @builder.io/qwik-city | Major | | eslint-plugin-qwik | Major |

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

intellix commented 2 weeks ago

just out of curiosity, what's the problem with TSM? was thinking that it looked promising for some of our other projects

JerryWu1234 commented 2 weeks ago

just out of curiosity, what's the problem with TSM? was thinking that it looked promising for some of our other projects

”because tsm doesn't support its import map.“ this can answer your question ?

wmertens commented 2 weeks ago

@intellix https://github.com/vfile/vfile/issues/90#issuecomment-2242667474

JerryWu1234 commented 1 week ago

@wmertens I tested pnpm run build.core and pnpm run build.full looks greet. and quick than ever

pkg-pr-new[bot] commented 1 week ago

Open in Stackblitz

npm i https://pkg.pr.new/@builder.io/qwik@6877
npm i https://pkg.pr.new/@builder.io/qwik-city@6877
npm i https://pkg.pr.new/eslint-plugin-qwik@6877
npm i https://pkg.pr.new/create-qwik@6877

commit: 5400426

JerryWu1234 commented 1 week ago

image

@wmertens I don't how to fix it, and other it looks greet