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
Replace Bluebird with native promises and async/await #7171
Comments
@richarddd great thanks for putting light on that.
We're aware of that issue, and want to do that shift soon. Still it'll require drop of support for Node.js v6+, and that's technically breaking semver wise, so we hold it until v2.0.0 release.
This best if done with async/await (otherwise it's doubled effort)
We can do that now. I would start by replacing those that can be easily fill in by native counterparts (as |
@richarddd I've updated this issue to be strictly about refactor to native promises and async/await, I've also outlined a proposal on how we can approach it |
@medikoo if we have a breakdown on how we can approach it probably I can pitch in, if that's okay. |
@muskeinsingh it'll simply be about refactoring all files that use Still at this point we need to wait for release of v2 first (and realization of #7362). We can assume this task is blocked until then |
@medikoo sure. I'll be waiting for an update on that! |
As we've dropped Node.js v6 dependency we can handle this now. I've split this into three issues. Please see updated top level description for details |
Well, this a rather big one and should be done in steps and with caution, preferably one file at the time.
I have been working with the framework for some time now and when you start digging into the core there are a lot of "old-ways" of doing stuff.
Proposed solution:
I divided this task into thee different issues, which should be handled in order
BbPromise.config
callsBbPromise
occurrences withPromise
The text was updated successfully, but these errors were encountered: