Skip to content
This repository has been archived by the owner on Jun 25, 2020. It is now read-only.

Add printfilename option #8

Merged
merged 1 commit into from
Jan 27, 2017
Merged

Add printfilename option #8

merged 1 commit into from
Jan 27, 2017

Conversation

di
Copy link
Collaborator

@di di commented May 18, 2016

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
Copy link

coveralls commented May 18, 2016

Coverage Status

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

@di
Copy link
Collaborator Author

di commented Jul 6, 2016

@sk- Any chance on this getting merged?

@alanhdu
Copy link

alanhdu commented Oct 27, 2016

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-?

Copy link
Owner

@sk- sk- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, sorry for the delay.

@sk- sk- merged commit eaf557e into sk-:master Jan 27, 2017
@di di deleted the add-printfilename branch January 30, 2017 02:39
@di
Copy link
Collaborator Author

di commented Jan 30, 2017

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

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

@sk-
Copy link
Owner

sk- commented Feb 2, 2017

@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
Copy link
Collaborator Author

di commented Feb 2, 2017

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

@di di mentioned this pull request Mar 30, 2017
@di
Copy link
Collaborator Author

di commented Mar 30, 2017

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

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants