search

sk- / html-linter

HTML5 Linter based on Google Style Guide
Apache License 2.0
7 stars 6 forks source link

Add printfilename option #8

Closed di closed 7 years ago

di commented 8 years ago

This PR adds the printfilename flag, which includes the filename which produced the error in the resulting error message (a la https://www.gnu.org/prep/standards/html_node/Errors.html)

This is useful when using html-linter with multiple input files.

For example:

$ html_lint.py --printfilename test/data/more_invalid.html
test/data/more_invalid.html:2:4: Error: Document Type: Do not close void elements: Remove the trailing "/" from the br tag.
coveralls commented 8 years ago

Coverage Status

Coverage increased (+0.003%) to 99.471% when pulling 9b3a937520e2d6e08aa3614b1c1cbecd8c78dba7 on di:add-printfilename into ab6ac074a84a11533ee8c1378f989fe676f8a63d on sk-:master.

di commented 8 years ago

@sk- Any chance on this getting merged?

alanhdu commented 8 years ago

FWIW, I would love to see this PR merged and released. It's a life-saver for linting dozens of files at once. Do you think that's possible @sk-?

di commented 7 years ago

Thanks @sk-. Are you planning a release anytime soon?

If this package needs an additional maintainer, I'd be happy to help.

sk- commented 7 years ago

@di Dustin, that would be awesome. I sent you an invite to be a collaborator on the project. Could you send me your Pypi username so I can add you as a maintainer of the package.

For travis I think it should be automatic once you have collaborator access to the project.

di commented 7 years ago

@sk- Awesome, thanks! My username is the same: https://pypi.org/user/di/

di commented 7 years ago

@alanhdu This has been released: https://pypi.org/project/html-linter/0.4.0/