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

Need Promise.reject(value) and Promise.resolve(value) instead of Promise.wrap() #501

Closed
eonarheim opened this issue Aug 15, 2015 · 0 comments · Fixed by #682
Closed

Need Promise.reject(value) and Promise.resolve(value) instead of Promise.wrap() #501

eonarheim opened this issue Aug 15, 2015 · 0 comments · Fixed by #682
Assignees
Labels
api change Breaking api change good first issue good for people new to open source and Excalibur
Milestone

Comments

@eonarheim
Copy link
Member

No description provided.

@eonarheim eonarheim added good first issue good for people new to open source and Excalibur on-deck labels Aug 15, 2015
@eonarheim eonarheim added this to the vNext milestone Aug 15, 2015
@jedeen jedeen removed the on-deck label Aug 19, 2015
@jedeen jedeen modified the milestones: 0.7 Release, vNext Aug 19, 2015
@jedeen jedeen modified the milestones: 0.7 Release, 0.8 Release Mar 12, 2016
@jedeen jedeen added the api change Breaking api change label Aug 21, 2016
@kamranayub kamranayub self-assigned this Nov 8, 2016
kamranayub added a commit that referenced this issue Nov 8, 2016
Closes #501

## Changes:

- Add `ex.Promise.resolve`
- Add `ex.Promise.reject`
- Deprecated `ex.Promise.wrap`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api change Breaking api change good first issue good for people new to open source and Excalibur
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants