-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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: commit types #9862
chore: commit types #9862
Conversation
To ensure that changes to code/types doesn't result in unwanted changes in type generation, or that bumps to dts-buddy don't cause unwanted regressions, we're checking in the generated types. Types should be committed as-is (don't format it with prettier!). CI is enhanced to check that git sees no changed files after generating the types, which would mean types have changed.
|
apart from the renaming of and again i'd suggest to split the checks into separate named items so that contributors (and us) can figure out faster which check has failed. |
Could you add those changes? You're probably more familiar with that syntax than I am. It also needs a change in the the corresponding Svelte 5 PR |
Did the changes except |
approved. the check steps could be made optional to be able to see multiple fails at once, but that can be in a different PR as well. There is a lint error that reads like some type issue, no clue about the macos fail. |
To ensure that changes to code/types doesn't result in unwanted changes in type generation, or that bumps to dts-buddy don't cause unwanted regressions, we're checking in the generated types. Types should be committed as-is (don't format it with prettier!). CI is enhanced to check that git sees no changed files after generating the types, which would mean types have changed.
This fails right now because types are wrong due to a regression from a
dts-buddy
bump. As such, no Svelte release should happen until this is fixed!Svelte 5 rewrite
Please note that the Svelte codebase is currently being rewritten for Svelte 5. Changes should target Svelte 5, which lives on the default branch (
main
).If your PR concerns Svelte 4 (including updates to svelte.dev.docs), please ensure the base branch is
svelte-4
and notmain
.Before submitting the PR, please make sure you do the following
feat:
,fix:
,chore:
, ordocs:
.Tests and linting
pnpm test
and lint the project withpnpm lint