-
Notifications
You must be signed in to change notification settings - Fork 385
Open
Open
Copy link
Labels
built-in pluginRelated to a built-in pluginRelated to a built-in plugincustom pluginRelated to a custom pluginRelated to a custom plugindeprecationSomething is/should be deprecatedSomething is/should be deprecatedinternaltypescriptRelated to TypescriptRelated to Typescript
Description
Hey 👋
Nowadays, we have TypeScript's satisfies AllProps<T> to achieve the same effect requireAllProps<T> has - ensuring a transormer visitor returns a node with all of it's properties. No runtime code needed.
This requires bumping the minimum TypeScript version to at least 4.9.
According to brief unreliable check, this could reduce ~1.4kb in minified size. Small saving alone, but with other things we could really make Kysely more lightweight, while keeping the same feature set AND staying as maintainable.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
built-in pluginRelated to a built-in pluginRelated to a built-in plugincustom pluginRelated to a custom pluginRelated to a custom plugindeprecationSomething is/should be deprecatedSomething is/should be deprecatedinternaltypescriptRelated to TypescriptRelated to Typescript