-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat: allow generated tsconfig to be modified #8606
Conversation
🦋 Changeset detectedLatest commit: f991ec3 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 |
I believe this won't be necessary with TypeScript 5 due out in March because if I'm remembering correctly it will allow you to extend from multiple tsconfigs. I feel like that might be the preferable solution though it would require making users wait two months. |
Interesting. I guess it comes down to whether there are other things beyond |
If the user has to change things I think that's fine. They can decide whether they want to upgrade and if they care about extending multiple tsconfigs then they can do that. I think it would become more problematic if there were things in SvelteKit that needed to be changed to allow you to run TypeScript 5. As far as I can tell, thus far we're probably safe and SvelteKit should work with TS 4 or 5: https://github.com/microsoft/TypeScript/wiki/API-Breaking-Changes. |
The question is 'aside from |
Marked this as draft because it probably makes sense to wait until TypeScript 5 if Until then, there's a workaround, so no-one is blocked on this even if it's a little inconvenient: manually copy the monorepo config into packages. |
Clarifying question, is this related to this typescript note from the (experimental)
Mentioning because I've recently been modifying the default I noticed that, even when |
@baisong that's unrelated to this PR/issue — this is purely about the contents of |
I'd be curious if this feature would solve an issue we're currently having. When extending the
We've tried adding It seems like this feature would allow people to get around issues like this in a more elegant way. |
Yep, that seems like a good enough reason! Marking this PR ready |
closes #6868, closes #8237
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:
Tests
pnpm test
and lint the project withpnpm lint
andpnpm check
Changesets
pnpm changeset
and following the prompts. Changesets that add features should beminor
and those that fix bugs should bepatch
. Please prefix changeset messages withfeat:
,fix:
, orchore:
.