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

Use native Promises instead of Q #681

Closed
raphinesse opened this issue Sep 6, 2018 · 0 comments · Fixed by #710
Closed

Use native Promises instead of Q #681

raphinesse opened this issue Sep 6, 2018 · 0 comments · Fixed by #710
Assignees
@raphinesse
Labels
enhancement

Comments

@raphinesse
Copy link
Contributor

raphinesse commented Sep 6, 2018 •
edited

We have to do this top down in our dependency graph (see below). cordova-cli and cordova-plugman are already Q-less. So lib is ready to go next.

Cordova Tooling dependency graph
raphinesse added a commit to raphinesse/cordova-lib that referenced this issue Sep 6, 2018
@raphinesse
Remove unused npm utility functions
03f1054 
This also removes usage of Q-specific functionality (see apache#681)
@raphinesse raphinesse mentioned this issue Sep 6, 2018
Merged
raphinesse added a commit that referenced this issue Sep 6, 2018
@raphinesse
Remove unused npm utility functions (#683)
d569393 
This also removes usage of Q-specific functionality (see #681)
This was referenced Sep 8, 2018
Merged
Open
Closed
@raphinesse raphinesse mentioned this issue Sep 29, 2018
Merged
@raphinesse raphinesse self-assigned this Sep 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@raphinesse raphinesse

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Drop dependency on Q, use native promises raphinesse/cordova-lib
1 participant
@raphinesse