Conversation
🦋 Changeset detectedLatest commit: 7884c0e 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 |
|
Ah, wait. The switch to I also wonder if we can skip the |
|
Vite is considering only supporting ESM output for 3.0. It'd be nice if we could make that work |
|
Vite isn't involved in this part at all |
|
Can we merge this now in favor of unblocking some folk seeing as #4969 might take some time? |
|
yep, i was hoping the nft version would work, but there are some lingering issues to work out |
fixes #4933. I don't truly understand why this was failing, but reverting to
cjsrather thanesmseems to fix it. If it turns out that other apps needesmthen we can expose it as a config optionPlease don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm testand lint the project withpnpm lintandpnpm checkChangesets
pnpm changesetand following the prompts. All changesets should bepatchuntil SvelteKit 1.0