sk- / html-linter Goto Github PK

To use this script with CI tools it would be useful if it could exit with a non zero status in case it detects errors.

A field like <label spellcheck="false">Example Label</label> will raise the following error:

241:173: Error: Boolean Attributes: Boolean attributes define their value based on the presence or abscence of the attribute: Change 'spellcheck="false"' to just 'spellcheck'.

First, this is wrong because spellcheck="false" is not the same as just spellcheck, however the documentation for the spellcheck attribute says:

This attribute is an enumerated one and not a Boolean one. This means that the explicit usage of one of the values true or false is mandatory and that a shorthand like <label spellcheck>Example Label</label> is not allowed. The correct usage is <label spellcheck="true">Example Label</label>.

So this error should not be raised for spellcheck or any similar enumerated field.

For files with handlebars templates like <script id="template" type="text/x-handlebars-template"> the following error is shown:

line 22, col 1: Error: Separation of concerns: Javascript should be defined in its own file: Move the contents of this tag to its own JS file.

Please create tags in GitHub for each release.

I'd be awesomely delighted by a timely release since the current one doesn't work with Python 3.5 but you apparently fixed that :)

Cheers!

The tests rely on the data, but it is missing from the source tarball

I just tried to launch the command with --disable=indentation but indentation errors are still showing up.

Please include LICENSE in the source tarball