mocha
mocha copied to clipboard
Use standard Promise.allSettled instead polyfill
Description of the Change
We've introduced @ungap/promise-all-settled in #4476. And Node.js support Promise.allSettled in Node 12.9.0.
So, we don't need polyfill anymore because we support Node 14+.
Why should this be in core?
We don't need polyfill anymore.
Benefits
dependency is reduced.
Possible Drawbacks
N/A
Coverage decreased (-0.002%) to 94.326% when pulling c557d79a382044bbb87f8023e01e26d730548844 on use-builtin-allsettled into 84b2f846148b180d6e1af088f77358a85c81d1ba on master.
@outsideris I guess this PR makes sense.
One of your links points to a korean page. I couldn't find any docs about Node supporting Promise.allSettled(). Can you help, please?
Oops! Sorry for korea link. https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise/allSettled
And you can see 12.9.0 release note.