Skip to content

Deprecate Thenable and alias to PromiseLike#239

Merged
aeschli merged 1 commit intomicrosoft:mainfrom
remcohaszing:deprecate-thenable
Aug 26, 2024
Merged

Deprecate Thenable and alias to PromiseLike#239
aeschli merged 1 commit intomicrosoft:mainfrom
remcohaszing:deprecate-thenable

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
Collaborator

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
@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