Allow for optionally disabling the title tag

[gkjohnson]

Changes
This PR wraps the title tag in the default template with a conditional that will only add the title tag if htmlWebpackPlugin.options.title is not equal to null or undefined.

Use Case
I'm building a project with a loader that injects HTML into the page, including the title tag in some places, and am using html-wepback-plugin to auto-generate a basic index.html page with that script included. It works fantastically except that my injected title tag isn't being respected because there is already one embedded by this plugin. This change allows you to optionally disable the addition of that title tag so an injected one is respected.

Other Questions
I'm consistently getting 11 tests failing on my local installation even with no changes to any files and after I've run yarn link and yarn link html-webpack-plugin in the directory. Is this expected? Is there something else I'm supposed to do?

Thanks!
Garrett

[jantimon]

wouldn't it be better to use a custom template without the title for your use case?

[gkjohnson]

I definitely could, but it seemed like such a simple thing and was surprised I couldn't disable the auto-inclusion of the title tag by default. It seems like the built in template should be (or be able to be) the most bare-bones page possible so your scripts or other imports can do the work if you want, so I thought I'd suggest the modification here.

[lock]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.