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

Convert all callback functions into promises and use async/await to deal with them #353

Closed
rafaelgomesxyz opened this issue Dec 31, 2017 · 4 comments

Comments

@rafaelgomesxyz
Copy link
Owner

This will make the code much cleaner and should also improve performance.

@rafaelgomesxyz rafaelgomesxyz changed the title Convert all callback functions in the script into promises and use async/await to deal with them Convert all callback functions into promises and use async/await to deal with them Dec 31, 2017
rafaelgomesxyz pushed a commit that referenced this issue Jan 1, 2018
@rafaelgomesxyz rafaelgomesxyz added this to the 7.12.1 milestone Jan 1, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 2, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 5, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 6, 2018
@rafaelgomesxyz rafaelgomesxyz modified the milestones: 7.12.1, 7.12.2 Jan 7, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 7, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 9, 2018
@rafaelgomesxyz rafaelgomesxyz modified the milestones: 7.12.2, 7.12.3 Jan 9, 2018
@rafaelgomesxyz rafaelgomesxyz modified the milestones: 7.13.0, 7.13.1 Jan 13, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 13, 2018
rafaelgomesxyz pushed a commit that referenced this issue Jan 24, 2018
@rafaelgomesxyz rafaelgomesxyz modified the milestones: 7.13.1, 7.14.0 Feb 3, 2018
rafaelgomesxyz pushed a commit that referenced this issue Feb 4, 2018
@gizellos
Copy link

gizellos commented Feb 5, 2018

Do you have to rewrite all the script because of it?

@rafaelgomesxyz
Copy link
Owner Author

Yes

@gizellos
Copy link

gizellos commented Feb 5, 2018

Please explain me what it does.

@rafaelgomesxyz
Copy link
Owner Author

Like I said in the first comment it should make the code cleaner and improve performance. But I'm also doing a lot of other improvements in this issue.

rafaelgomesxyz pushed a commit that referenced this issue Feb 8, 2018
rafaelgomesxyz pushed a commit that referenced this issue Mar 4, 2018
@rafaelgomesxyz rafaelgomesxyz removed this from the 7.14.0 milestone Mar 4, 2018
@rafaelgomesxyz rafaelgomesxyz added this to the 7.16.2 milestone Mar 4, 2018
@rafaelgomesxyz rafaelgomesxyz removed this from the 7.16.5 milestone Mar 29, 2018
rafaelgomesxyz pushed a commit that referenced this issue May 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants