Skip to content

v2.3.0 changes addressing issue 27 #34

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

Merged
merged 6 commits into from
Aug 12, 2020
Merged

v2.3.0 changes addressing issue 27 #34

merged 6 commits into from
Aug 12, 2020

Conversation

AntonioRedondo
Copy link
Collaborator

@AntonioRedondo AntonioRedondo commented Aug 4, 2020

These changes address issue #27:

  • BREAKING CHANGE: On the webpack configuration file the svgoConfig option must now go inside HtmlWebpackInlineSVGPlugin({}) instead of HtmlWebpackPlugin({}). An error is thrown at webpack build time otherwise.
  • The defaults for the svgo module aren't hardcoded anymore and –excepting the cleanupIDs option– the defaults are now set by the own module svgo and not by html-webpack-inline-svg-plugin.
  • package.json dependencies have been updated to latest versions.
  • Small corrections on README.md.

… option must now go inside `HtmlWebpackInlineSVGPlugin({})` instead of `HtmlWebpackPlugin({})`. An error in console is written at build time otherwise.

* The defaults for the `svgo` module aren't hardcoded anymore and –excepting the `cleanupIDs` option– the defaults are now set by the own module `svgo` and not `html-webpack-inline-svg-plugin`.

Tests not added yet
@AntonioRedondo AntonioRedondo requested a review from theGC August 4, 2020 07:45
@AntonioRedondo AntonioRedondo self-assigned this Aug 4, 2020
@AntonioRedondo AntonioRedondo merged commit d7aff06 into master Aug 12, 2020
@AntonioRedondo
Copy link
Collaborator Author

Merging PR. 🤜 Comments still welcome.

@AntonioRedondo
Copy link
Collaborator Author

@theGC does the package get released on npm automatically? Or do you have to press the button manually?

@theGC
Copy link
Owner

theGC commented Aug 12, 2020

it auto publishes to npm, looks like v2.3.0 is live
https://www.npmjs.com/package/html-webpack-inline-svg-plugin

@AntonioRedondo AntonioRedondo deleted the i27 branch August 1, 2021 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants