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

Update readme.md to indicate that this project is a ponyfill by default #68

Open
MatthewAry opened this issue Jan 31, 2020 · 1 comment

Comments

@MatthewAry
Copy link

MatthewAry commented Jan 31, 2020

The readme says

It's recommended to use this library in the form of a ponyfill, which doesn't inflict modifications of the global object.

The index.js file of the project does the ponyfill work already. Telling us to use the library in the form of a ponyfill is misleading, it should say something like "This library acts as a ponyfill by default, so as to not inflict modifications to the global object."

@acontreras89
Copy link

Can I ask why it is preferred to use this as a ponyfill instead of a polyfill? What are the benefits?

I was wondering but I did not want to open a new issue for such a (maybe silly) question.

cc/ @que-etc (sorry 🙏 )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants