Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add gnu custom output format #86

Merged
merged 1 commit into from
Oct 25, 2015
Merged

Add gnu custom output format #86

merged 1 commit into from
Oct 25, 2015

Conversation

hail2u
Copy link
Contributor

@hail2u hail2u commented Oct 15, 2015

This commit adds a custom output format that defined in GNU coding
standard
like this:

test.html:1.1: error: Doctype must be declared first.
test.html:5.35: error: Tag must be paired, missing: [ </p> ], start tag match failed [ <p> ] on line 5.

With this output format, text editors (such as Vim) can easily read
output from HTMLHint.

Sorry for lacking test case (I cannot find how to test bin options in this package).

@yaniswang
Copy link
Contributor

this style is named unix?

http://eslint.org/docs/user-guide/formatters/#unix

@yaniswang
Copy link
Contributor

image

@hail2u
Copy link
Contributor Author

hail2u commented Oct 15, 2015

Yes, ESLint and JSHint call this unix. Historically, this type of format calls gnu-emacs on HTML Tidy. I prefer latter, but unix is ok.

This commit adds a custom output format that defined in [GNU coding
standard][1] like this:

    test.html:1:1: error: Doctype must be declared first.
    test.html:5:35: error: Tag must be paired, missing: [ </p> ], start tag match failed [ <p> ] on line 5.

With this output format, text editors (such as Vim) can easily read
output from HTMLHint.

[1]: https://www.gnu.org/prep/standards/standards.html#Errors
@hail2u
Copy link
Contributor Author

hail2u commented Oct 15, 2015

Update to unix, and fix indentations also.

yaniswang added a commit that referenced this pull request Oct 25, 2015
Add `gnu` custom output format
@yaniswang yaniswang merged commit c9e2152 into htmlhint:master Oct 25, 2015
@hail2u hail2u deleted the gnu branch October 11, 2016 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants