Merge pull request #13 from Autre31415/default-tweak

update validator and switch default rules to standards

Author: kethinov

index.js

@@ -22,22 +22,7 @@ module.exports = (app, params) => {
 
   // default html-validate rules to use when none are passed
   const defaultRules = {
-    extends: ['html-validate:recommended'], // use the project's recommended rules, except...
-    rules: { // the defaults have some problems, so we're changing them
-      'attr-case': [ // has to do with permitted case (uppercase or lowercase) for attributes
-        'error',
-        {
-          rules: {
-            style: 'lowercase', // forces lowercase attributes
-            ignoreForeign: true // except in cases of non-HTML elements, e.g. <svg viewBox="0 0 100 100">
-          }
-        }
-      ],
-      'attr-quotes': 'off', // allows single or double quotes
-      'no-conditional-comment': 'off', // conditional comments are still useful sometimes and they're technically valid HTML, so there's no reason to have this rule
-      'no-trailing-whitespace': 'off', // linters should enforce this, not validators
-      'prefer-button': 'off' // this rule incorrectly disallows <input type="submit"> https://gitlab.com/html-validate/html-validate/-/issues/90
-    }
+    extends: ['html-validate:standard']
   }
 
   /**

package-lock.json

@@ -25,7 +25,7 @@
     "app-root-dir": "~1.0.2",
     "es6-template-strings": "~2.0.1",
     "fs-extra": "~9.0.0",
-    "html-validate": "~2.22.0",
+    "html-validate": "^2.23.0",
     "prismjs": "~1.20.0",
     "tamper": "~1.1.0"
   },

package.json

@@ -193,21 +193,6 @@ test('use defaults when validatorConfig is an empty object and no config file ex
   expressValidator(t.context.app, { rules: {} })
 
   t.deepEqual(config, {
-    extends: ['html-validate:recommended'], // use the project's recommended rules, except...
-    rules: { // the defaults have some problems, so we're changing them
-      'attr-case': [ // has to do with permitted case (uppercase or lowercase) for attributes
-        'error',
-        {
-          rules: {
-            style: 'lowercase', // forces lowercase attributes
-            ignoreForeign: true // except in cases of non-HTML elements, e.g. <svg viewBox="0 0 100 100">
-          }
-        }
-      ],
-      'attr-quotes': 'off', // allows single or double quotes
-      'no-conditional-comment': 'off', // conditional comments are still useful sometimes and they're technically valid HTML, so there's no reason to have this rule
-      'no-trailing-whitespace': 'off', // linters should enforce this, not validators
-      'prefer-button': 'off' // this rule incorrectly disallows <input type="submit"> https://gitlab.com/html-validate/html-validate/-/issues/90
-    }
+    extends: ['html-validate:standard']
   })
 })