Skip to content
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

types: group related and reduce potential inconsistencies #1539

Merged
merged 5 commits into from
May 26, 2021
Merged

types: group related and reduce potential inconsistencies #1539

merged 5 commits into from
May 26, 2021

Conversation

ignatiusmb
Copy link
Member

There's still a lot of manually written types which needs to be synchronized each time either one changes. This PR aims to hopefully reduce the need for that and just change them directly in .d.ts files.

@benmccann benmccann changed the title types: group related and reduce potential inconsitencies types: group related and reduce potential inconsistencies May 24, 2021
@benmccann benmccann merged commit eb96322 into sveltejs:master May 26, 2021
@ignatiusmb ignatiusmb deleted the types/reduce-inconsistencies branch May 26, 2021 16:42
sidharthv96 added a commit to sidharthv96/kit that referenced this pull request May 29, 2021
* 'master' of https://github.com/sveltejs/kit:
  Version Packages (next) (sveltejs#1543)
  type fixes for adapter-node and adapter-static (sveltejs#1578)
  Upgrade to Vite 2.3.3 (sveltejs#1580)
  fix: improve getRawBody parsing & handle error(s) (sveltejs#1528)
  create-svelte: add svelte-check for TS (sveltejs#1556)
  pass validated svelte config to adapters (sveltejs#1559)
  types: group related and reduce potential inconsistencies (sveltejs#1539)
  Use sveltekit tag on StackOverflow (sveltejs#1558)
  Fix create-svelte build-template script (sveltejs#1555)
  Remove err param from Polka .listen() callback (sveltejs#1550)
  bump: polka and sirv versions (sveltejs#1548)
  svelte-kit package (sveltejs#1499)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants