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
Removing babel-polyfill #268
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You may want to remove it from the tests so that it will test/ensure that the library can run without it. Otherwise we are just not including it as a dep and requiring the client to add it to their package.
Ref: https://github.com/Availity/availity-angular/blob/master/src/specs.js#L2
@KaseyPowers I removed the |
I think the removing of polyfill ended up being independent of the workflow v2 stuff so if you two aren't confident in that change It should be removable. |
@@ -1,27 +1,4 @@ | |||
import 'angular-shims-placeholder'; | |||
import ngModule from '../module'; | |||
|
|||
ngModule.requires.push('ng.shims.placeholder'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this even needed anymore?
Removing this to prevent core-js from always being added to projects