diff --git a/.eslintignore b/.eslintignore
index 80f61137d..24e7c619c 100644
--- a/.eslintignore
+++ b/.eslintignore
@@ -6,6 +6,3 @@ dist
# Website directory
website
-
-# eslint cannot lint itself
-.eslintrc.js
diff --git a/.eslintrc.js b/.eslintrc.js
index b37ed3f8b..4659da23c 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -1,46 +1,58 @@
module.exports = {
- extends: [
- 'eslint:recommended',
- 'plugin:prettier/recommended',
- 'plugin:@typescript-eslint/recommended',
- 'plugin:@typescript-eslint/recommended-requiring-type-checking',
- 'prettier/@typescript-eslint',
- ],
+ extends: ['eslint:recommended', 'plugin:prettier/recommended'],
env: {
- browser: true,
- amd: true,
+ es6: true,
node: true,
- mocha: true,
},
- parser: '@typescript-eslint/parser',
parserOptions: {
- ecmaVersion: 6,
sourceType: 'module',
- project: './tsconfig.lint.json',
- warnOnUnsupportedTypeScriptVersion: false,
},
- plugins: ['@typescript-eslint'],
- rules: {
- 'arrow-body-style': ['error'],
- 'no-template-curly-in-string': 'error',
- 'no-useless-concat': 'error',
- 'no-useless-escape': 'off',
- 'no-var': 'error',
- 'one-var': ['error', 'never'],
- 'prefer-arrow-callback': ['error', { allowNamedFunctions: true }],
- 'prefer-const': 'error',
- 'prefer-template': 'error',
- quotes: ['error', 'single', { avoidEscape: true }],
- 'template-curly-spacing': 'error',
- 'template-tag-spacing': 'error',
+ overrides: [
+ {
+ files: ['**/*.ts'],
+ extends: [
+ 'eslint:recommended',
+ 'plugin:@typescript-eslint/recommended',
+ 'plugin:@typescript-eslint/recommended-requiring-type-checking',
+ 'plugin:prettier/recommended',
+ ],
+ parser: '@typescript-eslint/parser',
+ parserOptions: {
+ ecmaVersion: 6,
+ sourceType: 'module',
+ project: './tsconfig.lint.json',
+ warnOnUnsupportedTypeScriptVersion: false,
+ },
+ plugins: ['@typescript-eslint'],
+ rules: {
+ 'arrow-body-style': ['error'],
+ 'no-template-curly-in-string': 'error',
+ 'no-useless-concat': 'error',
+ 'no-useless-escape': 'off',
+ 'no-var': 'error',
+ 'one-var': ['error', 'never'],
+ 'prefer-arrow-callback': ['error', { allowNamedFunctions: true }],
+ 'prefer-const': 'error',
+ 'prefer-template': 'error',
+ 'template-curly-spacing': 'error',
+ 'template-tag-spacing': 'error',
+ quotes: ['error', 'single', { avoidEscape: true }],
- '@typescript-eslint/adjacent-overload-signatures': 'off',
- '@typescript-eslint/ban-ts-comment': 'off',
- '@typescript-eslint/no-unsafe-assignment': 'off',
- '@typescript-eslint/no-unsafe-call': 'off',
- '@typescript-eslint/no-unsafe-member-access': 'off',
- '@typescript-eslint/no-unused-vars': 'off',
- '@typescript-eslint/no-var-requires': 'off',
- '@typescript-eslint/restrict-template-expressions': 'off',
- },
+ '@typescript-eslint/adjacent-overload-signatures': 'off',
+ '@typescript-eslint/ban-ts-comment': 'off',
+ '@typescript-eslint/no-unsafe-assignment': 'off',
+ '@typescript-eslint/no-unsafe-call': 'off',
+ '@typescript-eslint/no-unsafe-member-access': 'off',
+ '@typescript-eslint/no-unused-vars': 'off',
+ '@typescript-eslint/no-var-requires': 'off',
+ '@typescript-eslint/restrict-template-expressions': 'off',
+ },
+ },
+ {
+ files: ['**/*.spec.js'],
+ env: {
+ mocha: true,
+ },
+ },
+ ],
}
diff --git a/.github/dependabot.yml b/.github/dependabot.yml
index c5af001ac..cabdb7467 100644
--- a/.github/dependabot.yml
+++ b/.github/dependabot.yml
@@ -1,11 +1,19 @@
version: 2
updates:
- package-ecosystem: npm
- directory: "/"
+ directory: '/'
+ schedule:
+ interval: weekly
+ - package-ecosystem: npm
+ directory: '/website'
+ schedule:
+ interval: weekly
+ - package-ecosystem: npm
+ directory: '/website/plugins/docusaurus-plugin-hotjar'
schedule:
interval: weekly
versioning-strategy: increase
- package-ecosystem: github-actions
- directory: "/"
+ directory: '/'
schedule:
interval: weekly
diff --git a/.github/workflows/development.yml b/.github/workflows/development.yml
index a3a390b00..0e56751bd 100644
--- a/.github/workflows/development.yml
+++ b/.github/workflows/development.yml
@@ -13,13 +13,37 @@ on:
types: [opened, synchronize, reopened]
jobs:
+ lint:
+ runs-on: ubuntu-latest
+ steps:
+ - name: β¬οΈ Checkout
+ uses: actions/checkout@v2
+
+ - name: β Setup node ${{ matrix.node }}
+ uses: actions/setup-node@v2
+ with:
+ cache: npm
+
+ - name: π₯ Download deps
+ run: npm ci
+ env:
+ HUSKY_SKIP_INSTALL: true
+
+ - name: π§ͺ Run lint
+ run: npm run lint
+
build:
# ignore all-contributors PRs
if: ${{ !contains(github.head_ref, 'all-contributors') }}
runs-on: ubuntu-latest
+ needs: lint
strategy:
matrix:
- node: [14, 15]
+ node:
+ - 12
+ - 14
+ - 16
+ - 17
os: [ubuntu-latest, macOS-latest, windows-latest]
steps:
@@ -32,20 +56,16 @@ jobs:
uses: actions/checkout@v2
- name: β Setup node ${{ matrix.node }}
- uses: actions/setup-node@v2.4.1
+ uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node }}
+ cache: npm
- name: π₯ Download deps
- uses: bahmutov/npm-install@v1
- with:
- useLockFile: false
+ run: npm ci
env:
HUSKY_SKIP_INSTALL: true
- - name: π§ͺ Run lint
- run: npm run lint
-
- name: π Build
run: npm run build
env:
diff --git a/.github/workflows/labeler.yml b/.github/workflows/labeler.yml
index ca333641f..c2769f0c9 100644
--- a/.github/workflows/labeler.yml
+++ b/.github/workflows/labeler.yml
@@ -9,4 +9,4 @@ jobs:
steps:
- uses: actions/labeler@v3.0.2
with:
- repo-token: "${{ secrets.GITHUB_TOKEN }}"
+ repo-token: '${{ secrets.GITHUB_TOKEN }}'
diff --git a/.github/workflows/production.yml b/.github/workflows/production.yml
index 76fdae989..a73e2ec5b 100644
--- a/.github/workflows/production.yml
+++ b/.github/workflows/production.yml
@@ -12,13 +12,37 @@ on:
- '!all-contributors/**'
jobs:
+ lint:
+ runs-on: ubuntu-latest
+ steps:
+ - name: β¬οΈ Checkout
+ uses: actions/checkout@v2
+
+ - name: β Setup node ${{ matrix.node }}
+ uses: actions/setup-node@v2
+ with:
+ cache: npm
+
+ - name: π₯ Download deps
+ run: npm ci
+ env:
+ HUSKY_SKIP_INSTALL: true
+
+ - name: π§ͺ Run lint
+ run: npm run lint
+
build:
# ignore all-contributors PRs
if: ${{ !contains(github.head_ref, 'all-contributors') }}
runs-on: ubuntu-latest
+ needs: lint
strategy:
matrix:
- node: [14, 15]
+ node:
+ - 12
+ - 14
+ - 16
+ - 17
os: [ubuntu-latest, macOS-latest, windows-latest]
steps:
@@ -31,20 +55,16 @@ jobs:
uses: actions/checkout@v2
- name: β Setup node ${{ matrix.node }}
- uses: actions/setup-node@v2.4.1
+ uses: actions/setup-node@v2
with:
node-version: ${{ matrix.node }}
+ cache: npm
- name: π₯ Download deps
- uses: bahmutov/npm-install@v1
- with:
- useLockFile: false
+ run: npm ci
env:
HUSKY_SKIP_INSTALL: true
- - name: π§ͺ Run lint
- run: npm run lint
-
- name: π Build
run: npm run build
env:
@@ -59,8 +79,7 @@ jobs:
file: ./coverage/lcov.info
release:
- if:
- ${{ github.repository == 'htmlhint/HTMLHint' &&
+ if: ${{ github.repository == 'htmlhint/HTMLHint' &&
contains('refs/heads/master,refs/heads/beta,refs/heads/next,refs/heads/alpha',
github.ref) && github.event_name == 'push' }}
runs-on: ubuntu-latest
@@ -70,12 +89,12 @@ jobs:
uses: actions/checkout@v2
- name: β Setup node
- uses: actions/setup-node@v2.4.1
+ uses: actions/setup-node@v2
+ with:
+ cache: npm
- name: π₯ Download deps
- uses: bahmutov/npm-install@v1
- with:
- useLockFile: false
+ run: npm ci
env:
HUSKY_SKIP_INSTALL: true
diff --git a/.github/workflows/test-release.yml b/.github/workflows/test-release.yml
index 18902dbdd..43f11ed46 100644
--- a/.github/workflows/test-release.yml
+++ b/.github/workflows/test-release.yml
@@ -15,22 +15,18 @@ on:
jobs:
test-release:
runs-on: ubuntu-latest
- strategy:
- matrix:
- node: [14]
- os: [ubuntu-latest]
steps:
- name: β¬οΈ Checkout repo
uses: actions/checkout@v2
- name: β Setup node
- uses: actions/setup-node@v2.4.1
+ uses: actions/setup-node@v2
+ with:
+ cache: npm
- name: π₯ Download deps
- uses: bahmutov/npm-install@v1
- with:
- useLockFile: false
+ run: npm ci
env:
HUSKY_SKIP_INSTALL: true
@@ -56,4 +52,3 @@ jobs:
env:
GITHUB_TOKEN: ${{ secrets.GH_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
-
diff --git a/.prettierignore b/.prettierignore
index 0dc499740..b60541118 100644
--- a/.prettierignore
+++ b/.prettierignore
@@ -18,6 +18,7 @@ npm-debug.log*
.github/PULL_REQUEST_TEMPLATE.md
/website/playground
+/website/build
/website/.docusaurus
/website/plugins
/website/static
diff --git a/CHANGELOG.md b/CHANGELOG.md
index b5cbec70c..b3d3a0e93 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,18 +1,24 @@
-## [0.16.1](https://github.com/htmlhint/HTMLHint/compare/v0.16.0...v0.16.1) (2021-11-12)
+## [0.16.2](https://github.com/htmlhint/HTMLHint/compare/v0.16.1...v0.16.2) (2021-11-23)
### Bug Fixes
-* tagname-specialchars description ([#714](https://github.com/htmlhint/HTMLHint/issues/714)) ([4d702d8](https://github.com/htmlhint/HTMLHint/commit/4d702d8dfac3431fb71516b1ea3eb97950b63835))
+* clenaup non-functional typos ([#727](https://github.com/htmlhint/HTMLHint/issues/727)) ([98e45b9](https://github.com/htmlhint/HTMLHint/commit/98e45b9b2e1d7a4b0576d1944b1769c21b52b11e))
+
+## [0.16.2](https://github.com/htmlhint/HTMLHint/compare/v0.16.1...v0.16.2) (2021-11-23)
-## [0.16.1](https://github.com/htmlhint/HTMLHint/compare/v0.16.0...v0.16.1) (2021-11-12)
+### Bug Fixes
+
+* clenaup non-functional typos ([#727](https://github.com/htmlhint/HTMLHint/issues/727)) ([98e45b9](https://github.com/htmlhint/HTMLHint/commit/98e45b9b2e1d7a4b0576d1944b1769c21b52b11e))
+
+## [0.16.1](https://github.com/htmlhint/HTMLHint/compare/v0.16.0...v0.16.1) (2021-11-12)
### Bug Fixes
* tagname-specialchars description ([#714](https://github.com/htmlhint/HTMLHint/issues/714)) ([4d702d8](https://github.com/htmlhint/HTMLHint/commit/4d702d8dfac3431fb71516b1ea3eb97950b63835))
-# [0.16.0](https://github.com/htmlhint/HTMLHint/compare/v0.15.2...v0.16.0) (2021-10-29)
+## [0.16.0](https://github.com/htmlhint/HTMLHint/compare/v0.15.2...v0.16.0) (2021-10-29)
### Features
diff --git a/README.md b/README.md
index 02afd0269..50a771bd0 100644
--- a/README.md
+++ b/README.md
@@ -18,7 +18,7 @@
-
+
diff --git a/docs/user-guide/rules/input-requires-label.md b/docs/user-guide/rules/input-requires-label.md
index 8848740e5..63c5ecf1e 100644
--- a/docs/user-guide/rules/input-requires-label.md
+++ b/docs/user-guide/rules/input-requires-label.md
@@ -4,7 +4,7 @@ title: input-requires-label
keywords:
- input
- label
- - accessiblity
+ - accessibility
---
All [ input ] tags must have a corresponding [ label ] tag.
diff --git a/docs/user-guide/rules/src-not-empty.md b/docs/user-guide/rules/src-not-empty.md
index 87956d206..38482c9af 100644
--- a/docs/user-guide/rules/src-not-empty.md
+++ b/docs/user-guide/rules/src-not-empty.md
@@ -5,7 +5,7 @@ title: src-not-empty
Src of img(script, link) must set value.
-Emtpy of src will visit current page twice.
+Empty of src will visit current page twice.
Level: `error`
diff --git a/package-lock.json b/package-lock.json
index 05ce45392..292f72815 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -1,6 +1,6 @@
{
"name": "htmlhint",
- "version": "0.16.1",
+ "version": "0.16.2",
"lockfileVersion": 1,
"requires": true,
"dependencies": {
@@ -2054,9 +2054,9 @@
"dev": true
},
"@types/xml": {
- "version": "1.0.5",
- "resolved": "https://registry.npmjs.org/@types/xml/-/xml-1.0.5.tgz",
- "integrity": "sha512-h3PVM7waRi2UeoaY2BhpLGvettU/3vfCbsjXMV/9Ex5WjvIy82J8Qfp1xiPxM4kTSOLdFFpjRwQ7YY7XJeKBvg==",
+ "version": "1.0.6",
+ "resolved": "https://registry.npmjs.org/@types/xml/-/xml-1.0.6.tgz",
+ "integrity": "sha512-BCpp2oke88DwxJ/h748oLOQWdZ6k1Uv+aB9LZpyh/VhWqe4mE7X7ysNhF57cRPBs8/GyuUn1VRl+IlFdYsZPsA==",
"dev": true,
"requires": {
"@types/node": "*"
@@ -2452,9 +2452,9 @@
"dev": true
},
"async": {
- "version": "3.2.0",
- "resolved": "https://registry.npmjs.org/async/-/async-3.2.0.tgz",
- "integrity": "sha512-TR2mEZFVOj2pLStYxLht7TyfuRzaydfpxr3k9RpHIzMgw7A64dzsdqCxH1WJyQdoe8T10nDXd9wnEigmiuHIZw=="
+ "version": "3.2.2",
+ "resolved": "https://registry.npmjs.org/async/-/async-3.2.2.tgz",
+ "integrity": "sha512-H0E+qZaDEfx/FY4t7iLRv1W2fFI6+pyCeTw1uN20AQPiwqwM6ojPxHxdLv4z8hi2DtnW9BOckSspLucW7pIE5g=="
},
"asynckit": {
"version": "0.4.0",
diff --git a/package.json b/package.json
index 009248869..634374530 100644
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
{
"name": "htmlhint",
- "version": "0.16.1",
+ "version": "0.16.2",
"description": "The Static Code Analysis Tool for your HTML",
"repository": {
"type": "git",
@@ -30,8 +30,8 @@
"build:rollup": "npm run build:min && npm run build:unmin",
"build:min": "rollup -c --environment NODE_ENV:production --file dist/htmlhint.min.js",
"build:unmin": "rollup -c",
- "lint": "eslint . --ext .js,.ts --max-warnings 0",
- "lint:fix": "eslint --fix . --ext .js,.ts --max-warnings 0",
+ "lint": "prettier -c . && eslint . --ext .js,.ts --max-warnings 0",
+ "lint:fix": "npm run prettier && npm run lint -- --fix",
"test:coverage": "nyc npm run test"
},
"husky": {
@@ -59,7 +59,7 @@
},
"dependencies": {
"@types/node-fetch": "^2.5.12",
- "async": "3.2.0",
+ "async": "3.2.2",
"chalk": "4.1.0",
"commander": "5.1.0",
"glob": "7.1.7",
@@ -83,7 +83,7 @@
"@types/glob": "7.1.3",
"@types/node-fetch": "^2.5.12",
"@types/parse-glob": "3.0.29",
- "@types/xml": "1.0.5",
+ "@types/xml": "1.0.6",
"@typescript-eslint/eslint-plugin": "3.6.0",
"@typescript-eslint/parser": "3.10.1",
"commitizen": "^4.2.4",
diff --git a/parser-preset.js b/parser-preset.js
index 3df7c2652..9be580276 100644
--- a/parser-preset.js
+++ b/parser-preset.js
@@ -1,6 +1,6 @@
module.exports = {
parserOpts: {
- headerPattern: /^(\w*)\((\w*)\)\:\s(.*)$/,
+ headerPattern: /^(\w*)\((\w*)\):\s(.*)$/,
headerCorrespondence: ['type', 'scope', 'subject'],
},
}
diff --git a/src/cli/htmlhint.ts b/src/cli/htmlhint.ts
index 03c5327e6..e490604d7 100644
--- a/src/cli/htmlhint.ts
+++ b/src/cli/htmlhint.ts
@@ -174,7 +174,7 @@ function hintTargets(
})
}
-// load custom rles
+// load custom rules
function loadCustomRules(rulesdir: string) {
rulesdir = rulesdir.replace(/\\/g, '/')
if (existsSync(rulesdir)) {
@@ -215,7 +215,7 @@ function hintAllFiles(
formatter: Formatter
ruleset?: Ruleset
},
- onFinised: (result: {
+ onFinished: (result: {
targetFileCount: number
targetHintFileCount: number
targetHintCount: number
@@ -292,7 +292,7 @@ function hintAllFiles(
function checkAllHinted() {
if (isWalkDone && isHintDone) {
- onFinised({
+ onFinished({
targetFileCount: targetFileCount,
targetHintFileCount: targetHintFileCount,
targetHintCount: targetHintCount,
diff --git a/src/core/rules/tags-check.ts b/src/core/rules/tags-check.ts
index 272ecb42f..3f98206ef 100644
--- a/src/core/rules/tags-check.ts
+++ b/src/core/rules/tags-check.ts
@@ -146,7 +146,7 @@ export default {
redundantAttrs.forEach((attrName) => {
if (attrs.some((attr) => attr.name === attrName)) {
reporter.error(
- `The attr '${attrName}' is redundant for <${tagName}> and should be ommited.`,
+ `The attr '${attrName}' is redundant for <${tagName}> and should be omitted.`,
event.line,
col,
this,
diff --git a/test/cli/formatters/junit.spec.js b/test/cli/formatters/junit.spec.js
index 1954fe49d..607d0526d 100644
--- a/test/cli/formatters/junit.spec.js
+++ b/test/cli/formatters/junit.spec.js
@@ -1,7 +1,6 @@
const expect = require('expect.js')
const ChildProcess = require('child_process')
-const fs = require('fs')
const path = require('path')
describe('CLI', () => {
diff --git a/test/core.spec.js b/test/core.spec.js
index 3ee43ee35..2428136cc 100644
--- a/test/core.spec.js
+++ b/test/core.spec.js
@@ -62,7 +62,7 @@ describe('Core', () => {
expect(messages.length).to.be(0)
})
- it('Show formated result should not result in an error', () => {
+ it('Show formatted result should not result in an error', () => {
const code =
'tttttttttttttttttttttttttttttttttttt
tttttttttttttttttttttttttttttttttttttttttttttt'
const messages = HTMLHint.verify(code, {
diff --git a/test/rules/alt-require.spec.js b/test/rules/alt-require.spec.js
index 2af623258..177671ed1 100644
--- a/test/rules/alt-require.spec.js
+++ b/test/rules/alt-require.spec.js
@@ -2,12 +2,12 @@ const expect = require('expect.js')
const HTMLHint = require('../../dist/htmlhint.js').HTMLHint
-const ruldId = 'alt-require'
+const ruleId = 'alt-require'
const ruleOptions = {}
-ruleOptions[ruldId] = true
+ruleOptions[ruleId] = true
-describe(`Rules: ${ruldId}`, () => {
+describe(`Rules: ${ruleId}`, () => {
it('Img tag have empty alt attribute should not result in an error', () => {
const code = '' let messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) code = '
' messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(2) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(2) expect(messages[0].col).to.be(1) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(2) expect(messages[1].col).to.be(13) }) @@ -35,28 +35,28 @@ describe(`Rules: ${ruldId}`, () => { it('Set is false should not result in an error', () => { const code = '
' - ruleOptions[ruldId] = false + ruleOptions[ruleId] = false const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list should not result in an error', () => { const code = '
' - ruleOptions[ruldId] = ['testBox', 'tttAAA'] + ruleOptions[ruleId] = ['testBox', 'tttAAA'] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list with RegExp should not result in an error', () => { const code = '
' - ruleOptions[ruldId] = ['testBox', /bind:.*/] + ruleOptions[ruleId] = ['testBox', /bind:.*/] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) it('Set to array list with regex string should not result in an error', () => { const code = '
' - ruleOptions[ruldId] = ['testBox', '/\\[.*\\]/'] + ruleOptions[ruleId] = ['testBox', '/\\[.*\\]/'] const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(0) }) diff --git a/test/rules/attr-no-duplication.spec.js b/test/rules/attr-no-duplication.spec.js index eb5e2a614..7d80694aa 100644 --- a/test/rules/attr-no-duplication.spec.js +++ b/test/rules/attr-no-duplication.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-no-duplication' +const ruleId = 'attr-no-duplication' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute name been duplication should result in an error', () => { const code = 'bbb' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(12) }) diff --git a/test/rules/attr-no-unnecessary-whitespace.spec.js b/test/rules/attr-no-unnecessary-whitespace.spec.js index 39ad33585..8316121ac 100644 --- a/test/rules/attr-no-unnecessary-whitespace.spec.js +++ b/test/rules/attr-no-unnecessary-whitespace.spec.js @@ -2,12 +2,12 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-no-unnecessary-whitespace' +const ruleId = 'attr-no-unnecessary-whitespace' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute with spaces should result in an error', () => { const codes = [ '
', @@ -17,7 +17,7 @@ describe(`Rules: ${ruldId}`, () => { for (let i = 0; i < codes.length; i++) { const messages = HTMLHint.verify(codes[i], ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(5) } diff --git a/test/rules/attr-sort.spec.js b/test/rules/attr-sort.spec.js index 1bfcd68a9..f25cdad4f 100644 --- a/test/rules/attr-sort.spec.js +++ b/test/rules/attr-sort.spec.js @@ -9,7 +9,7 @@ ruleOptions[ruleId] = true describe(`Rules: ${ruleId}`, () => { it('Attribute unsorted tags must result in an error', () => { - const code = '' + const code = '' const messages = HTMLHint.verify(code, ruleOptions) diff --git a/test/rules/attr-unsafe-chars.spec.js b/test/rules/attr-unsafe-chars.spec.js index 60b5b0ffd..8c5285da1 100644 --- a/test/rules/attr-unsafe-chars.spec.js +++ b/test/rules/attr-unsafe-chars.spec.js @@ -2,18 +2,18 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-unsafe-chars' +const ruleId = 'attr-unsafe-chars' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value have unsafe chars should result in an error', () => { const code = 'Sud Web 2012' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) expect(messages[0].type).to.be('warning') diff --git a/test/rules/attr-value-double-quotes.spec.js b/test/rules/attr-value-double-quotes.spec.js index 3ae786fe1..0a35190a4 100644 --- a/test/rules/attr-value-double-quotes.spec.js +++ b/test/rules/attr-value-double-quotes.spec.js @@ -2,23 +2,23 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-double-quotes' +const ruleId = 'attr-value-double-quotes' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value closed by single quotes should result in an error', () => { const code = "" const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(14) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(1) expect(messages[2].col).to.be(24) }) diff --git a/test/rules/attr-value-not-empty.spec.js b/test/rules/attr-value-not-empty.spec.js index 57f083d23..d2fd6e840 100644 --- a/test/rules/attr-value-not-empty.spec.js +++ b/test/rules/attr-value-not-empty.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-not-empty' +const ruleId = 'attr-value-not-empty' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value have no value should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) expect(messages[0].type).to.be('warning') diff --git a/test/rules/attr-value-single-quotes.spec.js b/test/rules/attr-value-single-quotes.spec.js index e0da6cb05..e05203b41 100644 --- a/test/rules/attr-value-single-quotes.spec.js +++ b/test/rules/attr-value-single-quotes.spec.js @@ -2,23 +2,23 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-value-single-quotes' +const ruleId = 'attr-value-single-quotes' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Attribute value closed by double quotes should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(3) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(14) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(1) expect(messages[2].col).to.be(24) }) diff --git a/test/rules/attr-whitespace.spec.js b/test/rules/attr-whitespace.spec.js index e093b74ee..67edfc5a2 100644 --- a/test/rules/attr-whitespace.spec.js +++ b/test/rules/attr-whitespace.spec.js @@ -2,17 +2,17 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'attr-whitespace' +const ruleId = 'attr-whitespace' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { +describe(`Rules: ${ruleId}`, () => { it('Double spaces in attributes should result in an error', () => { const code = '' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) }) @@ -20,7 +20,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '
' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) }) @@ -28,7 +28,7 @@ describe(`Rules: ${ruldId}`, () => { const code = '
'
const messages = HTMLHint.verify(code, ruleOptions)
expect(messages.length).to.be(2)
- expect(messages[0].rule.id).to.be(ruldId)
+ expect(messages[0].rule.id).to.be(ruleId)
expect(messages[0].line).to.be(1)
})
})
diff --git a/test/rules/doctype-first.spec.js b/test/rules/doctype-first.spec.js
index 5e381a401..993c2d86e 100644
--- a/test/rules/doctype-first.spec.js
+++ b/test/rules/doctype-first.spec.js
@@ -2,17 +2,17 @@ const expect = require('expect.js')
const HTMLHint = require('../../dist/htmlhint.js').HTMLHint
-const ruldId = 'doctype-first'
+const ruleId = 'doctype-first'
const ruleOptions = {}
-ruleOptions[ruldId] = true
+ruleOptions[ruleId] = true
-describe(`Rules: ${ruldId}`, () => {
+describe(`Rules: ${ruleId}`, () => {
it('Doctype not be first should result in an error', () => {
const code = ''
const messages = HTMLHint.verify(code, ruleOptions)
expect(messages.length).to.be(1)
- expect(messages[0].rule.id).to.be(ruldId)
+ expect(messages[0].rule.id).to.be(ruleId)
expect(messages[0].line).to.be(1)
expect(messages[0].col).to.be(1)
})
diff --git a/test/rules/doctype-html5.spec.js b/test/rules/doctype-html5.spec.js
index cafdcd550..a90f51614 100644
--- a/test/rules/doctype-html5.spec.js
+++ b/test/rules/doctype-html5.spec.js
@@ -2,18 +2,18 @@ const expect = require('expect.js')
const HTMLHint = require('../../dist/htmlhint.js').HTMLHint
-const ruldId = 'doctype-html5'
+const ruleId = 'doctype-html5'
const ruleOptions = {}
-ruleOptions[ruldId] = true
+ruleOptions[ruleId] = true
-describe(`Rules: ${ruldId}`, () => {
+describe(`Rules: ${ruleId}`, () => {
it('Doctype not html5 should result in an error', () => {
const code =
''
const messages = HTMLHint.verify(code, ruleOptions)
expect(messages.length).to.be(1)
- expect(messages[0].rule.id).to.be(ruldId)
+ expect(messages[0].rule.id).to.be(ruleId)
expect(messages[0].line).to.be(1)
expect(messages[0].col).to.be(1)
expect(messages[0].type).to.be('warning')
diff --git a/test/rules/empty-tag-not-self-closed.spec.js b/test/rules/empty-tag-not-self-closed.spec.js
index 9c2b69e00..418933826 100644
--- a/test/rules/empty-tag-not-self-closed.spec.js
+++ b/test/rules/empty-tag-not-self-closed.spec.js
@@ -2,12 +2,12 @@ const expect = require('expect.js')
const HTMLHint = require('../../dist/htmlhint.js').HTMLHint
-const ruldId = 'empty-tag-not-self-closed'
+const ruleId = 'empty-tag-not-self-closed'
const ruleOptions = {}
-ruleOptions[ruldId] = true
+ruleOptions[ruleId] = true
-describe(`Rules: ${ruldId}`, () => {
+describe(`Rules: ${ruleId}`, () => {
it('The empty tag no closed should not result in an error', () => {
const code = '
'
const messages = HTMLHint.verify(code, ruleOptions)
@@ -18,11 +18,11 @@ describe(`Rules: ${ruldId}`, () => {
const code = '
'
const messages = HTMLHint.verify(code, ruleOptions)
expect(messages.length).to.be(2)
- expect(messages[0].rule.id).to.be(ruldId)
+ expect(messages[0].rule.id).to.be(ruleId)
expect(messages[0].line).to.be(1)
expect(messages[0].col).to.be(1)
expect(messages[0].type).to.be('error')
- expect(messages[1].rule.id).to.be(ruldId)
+ expect(messages[1].rule.id).to.be(ruleId)
expect(messages[1].line).to.be(1)
expect(messages[1].col).to.be(7)
expect(messages[1].type).to.be('error')
diff --git a/test/rules/head-require.spec.js b/test/rules/head-require.spec.js
index eddd646c4..10fe84d61 100644
--- a/test/rules/head-require.spec.js
+++ b/test/rules/head-require.spec.js
@@ -2,17 +2,17 @@ const expect = require('expect.js')
const HTMLHint = require('../../dist/htmlhint.js').HTMLHint
-const ruldId = 'head-script-disabled'
+const ruleId = 'head-script-disabled'
const ruleOptions = {}
-ruleOptions[ruldId] = true
+ruleOptions[ruleId] = true
-describe(`Rules: ${ruldId}`, () => {
+describe(`Rules: ${ruleId}`, () => {
it('External script in head should result in an error', () => {
const code = '

'
const messages = HTMLHint.verify(code, ruleOptions)
expect(messages.length).to.be(2)
- expect(messages[0].rule.id).to.be(ruldId)
+ expect(messages[0].rule.id).to.be(ruleId)
expect(messages[0].line).to.be(1)
expect(messages[0].col).to.be(26)
expect(messages[0].type).to.be('warning')
@@ -31,7 +31,7 @@ describe(`Rules: ${ruldId}`, () => {
'aaa>bbb< ccc
ddd\r\neee<' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(3) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(7) - expect(messages[1].rule.id).to.be(ruldId) + expect(messages[1].rule.id).to.be(ruleId) expect(messages[1].line).to.be(1) expect(messages[1].col).to.be(11) - expect(messages[2].rule.id).to.be(ruldId) + expect(messages[2].rule.id).to.be(ruleId) expect(messages[2].line).to.be(2) expect(messages[2].col).to.be(4) }) @@ -27,7 +27,7 @@ describe(`Rules: ${ruldId}`, () => { const code = 'Steinway & Sons
' const messages = HTMLHint.verify(code, ruleOptions) expect(messages.length).to.be(1) - expect(messages[0].rule.id).to.be(ruldId) + expect(messages[0].rule.id).to.be(ruleId) expect(messages[0].line).to.be(1) expect(messages[0].col).to.be(12) }) diff --git a/test/rules/src-not-empty.spec.js b/test/rules/src-not-empty.spec.js index 8e6a6a740..1fee585bc 100644 --- a/test/rules/src-not-empty.spec.js +++ b/test/rules/src-not-empty.spec.js @@ -2,13 +2,13 @@ const expect = require('expect.js') const HTMLHint = require('../../dist/htmlhint.js').HTMLHint -const ruldId = 'src-not-empty' +const ruleId = 'src-not-empty' const ruleOptions = {} -ruleOptions[ruldId] = true +ruleOptions[ruleId] = true -describe(`Rules: ${ruldId}`, () => { - it('Src be emtpy should result in an error', () => { +describe(`Rules: ${ruleId}`, () => { + it('Src be empty should result in an error', () => { const code = '