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.

Sign up for GitHub

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

Jump to bottom

Better typings for Promise executor, like #31117 #33062

Closed
wants to merge 2 commits into from
Closed

Better typings for Promise executor, like #31117 #33062

wants to merge 2 commits into from

Conversation

jablko
Copy link
Contributor

@jablko jablko commented Aug 23, 2019 •
edited
Loading

Fixes #27711

@jablko jablko force-pushed the patch-4 branch 2 times, most recently from 1f6d527 to c7827b5 Compare August 30, 2019 16:26
@jablko jablko force-pushed the patch-4 branch 3 times, most recently from a711904 to 246b533 Compare September 25, 2019 23:18
@jablko jablko force-pushed the patch-4 branch 2 times, most recently from b997d79 to 608ef2d Compare October 1, 2019 15:02
@jablko jablko mentioned this pull request Oct 1, 2019
Closed
@jablko jablko force-pushed the patch-4 branch 4 times, most recently from 5d2ccd3 to 916d02c Compare October 7, 2019 14:31
@jablko jablko force-pushed the patch-4 branch 2 times, most recently from 12caf59 to 718dedc Compare October 16, 2019 14:46
@jablko jablko force-pushed the patch-4 branch 2 times, most recently from 1e9068f to 5cbbca9 Compare October 19, 2019 18:34
@jablko jablko force-pushed the patch-4 branch 2 times, most recently from 631b917 to 95b9f3d Compare November 16, 2019 15:24
@jablko jablko mentioned this pull request Nov 18, 2019
Closed
@jablko jablko force-pushed the patch-4 branch 5 times, most recently from 92e90f8 to ad941f4 Compare November 28, 2019 14:51
@jablko jablko force-pushed the patch-4 branch 2 times, most recently from 76af7f9 to ecfc544 Compare January 25, 2020 20:23
@jablko jablko closed this Jan 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ExE-Boss ExE-Boss ExE-Boss approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Promise<Promise<T>> cannot exist in JS
2 participants
@jablko @ExE-Boss