Skip to content

fix AsyncDisposable usage erroring for older TypeScript versions. - #1507

Merged
igalklebanov merged 1 commit into
kysely-org:masterfrom
igalklebanov:fix-old-typescript-support
Jul 17, 2025
Merged

fix AsyncDisposable usage erroring for older TypeScript versions.#1507
igalklebanov merged 1 commit into
kysely-org:masterfrom
igalklebanov:fix-old-typescript-support

Conversation

@igalklebanov

@igalklebanov igalklebanov commented Jul 8, 2025

Copy link
Copy Markdown
Member

Hey 👋

closes #1499.

@vercel

vercel Bot commented Jul 8, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
kysely ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2025 0:58am

@pkg-pr-new

pkg-pr-new Bot commented Jul 8, 2025

Copy link
Copy Markdown

kysely_koa_example

npm i https://pkg.pr.new/kysely-org/kysely@1507

commit: a1e45f6

@igalklebanov
igalklebanov force-pushed the fix-old-typescript-support branch from 0e448cf to 50ab0ba Compare July 8, 2025 23:09
@igalklebanov
igalklebanov force-pushed the fix-old-typescript-support branch from 5977612 to 50ab0ba Compare July 8, 2025 23:19
@igalklebanov igalklebanov changed the title TBD fix AsyncDisposable usage erroring for older TypeScript versions. Jul 8, 2025
@igalklebanov igalklebanov added bug Something isn't working typescript Related to Typescript labels Jul 8, 2025
const steps: string[] = []

{
// @ts-ignore - `using` was only introduced in TS 5.2

@igalklebanov igalklebanov Jul 8, 2025

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we're excluding the entire file from the older version typechecks via a CI script.

Comment thread test/node/tsconfig.json
@@ -1 +1,11 @@
{"compilerOptions":{"target":"ES2022","lib":["ESNext"],"module":"CommonJS","outDir":"dist","skipLibCheck":true},"default":{"compilerOptions":{"target":"ES2022","lib":["ESNext"],"module":"CommonJS","outDir":"dist","skipLibCheck":true},"default":{"compilerOptions":{"target":"ES2022","lib":["ESNext"],"module":"CommonJS","outDir":"dist","skipLibCheck":true},"default":{"extends":"../../tsconfig-base.json","include":["src/**/*"],"compilerOptions":{"target":"ES2022","lib":["ESNext"],"module":"CommonJS","outDir":"dist","skipLibCheck":true}},"extends":"../../tsconfig-base.json","include":["src/**/*"],"exclude":["src/async-dispose.test.ts"]},"exclude":["src/async-dispose.test.ts"],"extends":"../../tsconfig-base.json","include":["src/**/*"]},"exclude":["src/async-dispose.test.ts","src/async-dispose.test.ts"],"extends":"../../tsconfig-base.json","include":["src/**/*"]} No newline at end of file

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

😨

...

polyfill AsyncDisposable.
@igalklebanov
igalklebanov force-pushed the fix-old-typescript-support branch from a77a74e to a1e45f6 Compare July 10, 2025 00:57
@igalklebanov
igalklebanov marked this pull request as ready for review July 17, 2025 08:10
@igalklebanov
igalklebanov merged commit 2f624c7 into kysely-org:master Jul 17, 2025
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working typescript Related to Typescript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Kysely does not work with Typescript below version 5.2 because of AsyncDisposable

1 participant