Skip to content

fix: improve syntax for drizzle config#1146

Merged
manuel3108 merged 5 commits into
sveltejs:mainfrom
sacrosanctic:irrelevant-junglefowl
Jun 26, 2026
Merged

fix: improve syntax for drizzle config#1146
manuel3108 merged 5 commits into
sveltejs:mainfrom
sacrosanctic:irrelevant-junglefowl

Conversation

@sacrosanctic

Copy link
Copy Markdown
Contributor

Description

A function that allows you to edit the generated tsconfig.json. You can mutate the config (recommended)
https://svelte.dev/docs/kit/configuration#typescript

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@changeset-bot

changeset-bot Bot commented Jun 21, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 7b0b82c

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

This PR includes changesets to release 2 packages
Name Type
sv Patch
@sveltejs/sv-utils Patch

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

@sacrosanctic

sacrosanctic commented Jun 21, 2026

Copy link
Copy Markdown
Contributor Author

Something weird happened with the snapshots.

@jycouet

jycouet commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Something weird happened with the snapshots.

Better like this?

@sacrosanctic

Copy link
Copy Markdown
Contributor Author

Nice, do we need line break for the final output or do we run formating in the CLI?

@jycouet

jycouet commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Nice, do we need line break for the final output or do we run formating in the CLI?

Where? 👀
We run format only if we have prettier there

@sacrosanctic

Copy link
Copy Markdown
Contributor Author

We have it as one line but prettier will format it to three lines.

@jycouet

jycouet commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

We have it as one line but prettier will format it to three lines.

we see it formatted in snapshots. I think it's good.

@jycouet jycouet self-requested a review June 22, 2026 14:06
@manuel3108 manuel3108 merged commit f85a0ea into sveltejs:main Jun 26, 2026
5 of 6 checks passed
@github-actions github-actions Bot mentioned this pull request Jun 26, 2026
@sacrosanctic sacrosanctic deleted the irrelevant-junglefowl branch June 26, 2026 06:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants