-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
html-webpack-plugin 4.x - Switching to Webpack 4 api #953
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.
Great work! enabling the strict mode is not far
lib/html-tags.js
Outdated
*/ | ||
|
||
/** | ||
* @type {string[]} - All html tag elements which must not contain innerHTML |
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.
Let typescript infer types if it can
lib/html-tags.js
Outdated
|
||
/** | ||
* Turn a tag definition into a html string | ||
* @param tagDefinition {HtmlTagObject} |
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.
order is wrong. types come first
/// <reference path="../typings.d.ts" /> | ||
/* eslint-enable */ | ||
/* | ||
* @file |
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.
To use JSDoc directives use JSDoc comment sections that start with /**
c1e57b5
to
1eb2575
Compare
Thanks for your fast and detailed feedback! |
Released as alpha: https://www.npmjs.com/package/html-webpack-plugin/v/4.0.0-alpha
|
BREAKING CHANGE: switch to loaderUtils getOptions
BREAKING CHANGE: Chunks aren't sorted anymore by default
BREAKING CHANGE: Template variable webpack was removed
BREAKING CHANGE: Dropped support for Webpack 1 - 3
Only transpile EJS style `<%= name %>`, Close #950 BREAKING CHANGE: Template strings inside templates are now disabled by default
BREAKING CHANGE: Remove selfClosingTag attribute
See https://www.w3.org/TR/html5/scripting-1.html#attr-script-type “The default, which is used if the attribute is absent, is "text/javascript"” BREAKING CHANGE: The mimetype information "text/javascript" is removed from all generated script tags
See https://developer.mozilla.org/en-US/docs/Web/Guide/HTML/Obsolete_things_to_avoid BREAKING CHANGE: The default meta utf-8 declaration was changed to <meta charset="utf-8"/>
Released as 4.0.0-beta.1 |
Hi! Thank you for preparing a new release. However it doesn't appear to be published/public?
|
Is available now - thank you :-) |
BREAKING: `contenthash` in now `templatehash`. Since webpack 4.3 `contenthash` is used by webpack. to avoid conflicts, it is renamed to `templatehash`
Update JSDoc
- types packages should not be installed for plugin users - adjusted bundles to current webpack@4 behavior
Merged and released as This version is also the release candidate. |
Waiting for |
Hey @Saravanan90 so far feedback is quite good. I would love to solve #1089 #1091 and #1072 before we release it as stable 4.x |
Eagerly awaiting for |
Bug Fixes
Code Refactoring
Features
BREAKING CHANGES
htmlWebpackPluginAlterChunks
hook. Changed the structure of theassets
argument for all hooks.{ entryName: string, path: string}
tags
New Event names
Performance improvements
Performance findings by different users for v4: