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
feat: replace instanceof Promise and support Promise/A+ #310
Conversation
@NoNameProvided Would you be able to take a look at this please when you have time? Thanks |
I am ok with support Promise/A+ compliant thenable but why we need another dependency because of one function? Could you please create e.g. new |
@vlapo Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrewwa-kainos Thnx for your effort. Could you please check my in-code comment?
Note: We have also new ValidatePromise
(merged it few moments ago). Could you rebase and also use your isPromise
method in this validator?
Thank you for your contribution :) |
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This fixes a problem found wherein "Promise" instances from ES5 were not recognised as "instanceof Promise" from ES2015.