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

Deprecate Thenable and alias to PromiseLike #239

Merged
merged 1 commit into from
Aug 26, 2024

Conversation

remcohaszing
Copy link
Contributor

@remcohaszing remcohaszing commented Jul 30, 2024

TypeScript has a builtin type for promise-like values: PromiseLike. Thenable attempts to achieve the same result, but it has some compatibility issues.

This change aliases Thenable to PromiseLike, to make it compatible with the rest of the TypeScript ecosystem.

Additionally this deprecates Thenable. There is no practical use case for it, but it is confusing to communicate something other than PromiseLike to users. All occurrences of Thenable were replaced with PromiseLike.

TypeScript has a builtin type for promise-like values: PromiseLike.
Thenable attempts to achieve the same result, but it has some
compatibility issues.

This change aliases Tnenable to PromiseLike, to make it compatible with
the rest of the TypeScript ecosystem.

Additionally this deprecates Thenable. There is no practical use case
for it, but it is confusing to communicate something other than
PromiseLike to users. All occurrences of Thenable were replaced with
PromiseLike.
@aeschli
Copy link
Contributor

aeschli commented Aug 26, 2024

Thanks @remcohaszing !

@vs-code-engineering vs-code-engineering bot added this to the August 2024 milestone Aug 26, 2024
@aeschli aeschli enabled auto-merge August 26, 2024 17:46
@aeschli aeschli merged commit 24678bd into microsoft:main Aug 26, 2024
2 checks passed
@remcohaszing remcohaszing deleted the deprecate-thenable branch August 26, 2024 18:20
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