Merge pull request #2 from swimmadude66/dev

Merge in readme fix, plus actual config type and updated package-lock

Author: Adam Yost

.npmignore

@@ -2,6 +2,7 @@ src/
 spec/
 node_modules/
 .vscode/
+.gitignore
 npm-debug.log
 tsconfig.json
 .travis.yml

.travis.yml

@@ -1,14 +1,13 @@
 language: node_js
 node_js:
   - "8"
-  - "9"
   - "10"
-  - "11"
   - "12"
 
 env:
   - WEBPACK_VERSION=4 HTML_PLUGIN_VERSION=3
   - WEBPACK_VERSION=4 HTML_PLUGIN_VERSION=next
+  # - WEBPACK_VERSION=5.0.0-alpha.12 HTML_PLUGIN_VERSION=next
 
 install: 
   - npm install

README.md

@@ -1,5 +1,5 @@
 # Webpack Nomodule Plugin
-_Assigns the nodmodule attribute to script tags injected by [Html Webpack Plugin](https://github.com/jantimon/html-webpack-plugin)_
+_Assigns the `nomodule` attribute to script tags injected by [Html Webpack Plugin](https://github.com/jantimon/html-webpack-plugin)_
 
 ## Configuration
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "webpack-nomodule-plugin",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "description": "a plugin for html-webpack-plugin to label certain chunks as legacy-only via the nomodule attribute",
   "main": "dist/plugin.js",
   "scripts": {

package.json

@@ -1,9 +1,14 @@
 import * as HtmlWebpackPlugin from 'html-webpack-plugin';
 import * as minimatch from 'minimatch';
 
+export type NoModuleConfig = {
+    filePatterns: string[];
+    // in case I want to add other optional configs later without breaking old uses
+}
+
 export class WebpackNoModulePlugin {
 
-    constructor(private _config: {filePatterns: string[]} = {filePatterns: []}) {
+    constructor(private _config: NoModuleConfig = {filePatterns: []}) {
 
     }
 
@@ -21,9 +26,9 @@ export class WebpackNoModulePlugin {
                             return cb(null, data);
                         }
                     )
-                } else if (HtmlWebpackPlugin && HtmlWebpackPlugin['getHooks']) {
+                } else if (HtmlWebpackPlugin && HtmlWebpackPlugin.getHooks) {
                     // html-webpack 4
-                    const hooks = (HtmlWebpackPlugin as any).getHooks(compilation);
+                    const hooks = HtmlWebpackPlugin.getHooks(compilation);
                     hooks.alterAssetTags.tapAsync(
                         'NoModulePlugin',
                         (data, cb) => {