Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(rollup): provide es5 and minified umd bundles #710

Merged
merged 1 commit into from
Apr 24, 2019

Conversation

jbedard
Copy link
Collaborator

@jbedard jbedard commented Apr 21, 2019

Closes #685

Would be nice to have better e2e tests for all output formats imo, but I think that's a separate task. Same with fixing the es5 sourcemaps...

Copy link
Collaborator

@kyliau kyliau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please also update rollup.spec.js to check the named outputs?

@jbedard
Copy link
Collaborator Author

jbedard commented Apr 22, 2019

Basically just add a golden file for the for the es5 UMD?

Copy link
Collaborator

@alexeagle alexeagle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@jbedard
Copy link
Collaborator Author

jbedard commented Apr 23, 2019

Updated to add the test + golden files for es5umd and min.es5umd.

Were there any naming preferences for something other then "es5umd"?

@kyliau kyliau merged commit d70cf62 into bazel-contrib:master Apr 24, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

rollup_bundle: provide es5 UMD bundle
4 participants