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

Promise.reject uses an Error instead of a string #4684

Merged
merged 1 commit into from
Mar 26, 2019

Conversation

theBadT
Copy link
Contributor

@theBadT theBadT commented Mar 24, 2019

Closes #4560

Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

It should reject with a new Error, instead of the Error.

Signed-off-by: Thomas Drosdzoll <thomas.drosdzoll@gmx.net>

added new to Error instation
Copy link
Contributor

@kittaakos kittaakos left a comment

Choose a reason for hiding this comment

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

It looks good to me. Thank you for the patch! 👍

@kittaakos kittaakos merged commit 68795ec into eclipse-theia:master Mar 26, 2019
@theBadT theBadT deleted the 4560-reject-promise branch March 26, 2019 17:46
kittaakos referenced this pull request Apr 17, 2020
Signed-off-by: Vladyslav Zhukovskyi <vzhukovs@redhat.com>
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.

4 participants