-
Notifications
You must be signed in to change notification settings - Fork 594
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
Missing polyfill for Promise.allSettled #354
Comments
I just checked the bundle size for |
Wow thats pretty large. |
The reason why Here is a polyfill in 218 bytes. Would love to see this in es6-promise, which is my favorite promise polyfill ❤️ |
This project is named es6-promise, and es6 doesn't have all-settled, or other new features. If there is enough demand, we can create a different polyfil. But it is unclear if adding it here, given the name, makes sense. Thoughts? |
I don't think much people rely on the fact that ES2015 doesn't contain I guess changing this project name isn't an option? |
As a long-time user of this package, it never occured to me that it its features could be scoped to ES2015. I guess it's because we've been using "ES6" as an inaccurate alias for "the time when we got new JavaScript features" for so long. In any case, other packages I use have started to use |
I implemented this method https://github.com/WormGirl/core-promise |
Now that
Promise.allSettled
reached stage 4 of the specification process, would it make sense to add a polyfill for it to this project?Or would you rather recommend to add a separate shim for it, something like es-shims/Promise.allSettled ? IMO that's less convenient, but creating yet another implementation also sounds wrong.
The text was updated successfully, but these errors were encountered: