We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Using Atom's linter-jshint, I'm getting JSHint errors on the wrong lines for inline JavaScript in the HTML body:
HTML source: https://gist.github.com/rmzelle/e5bc6f1a30113f69db9c
This is with Atom 1.3.2, linter 1.11.3, and linter-jshint 1.2.2 (with "Lint Inline Java Script" checked).
The HTML validates according to https://validator.w3.org/ and it looks like embedding JS in the body is indeed allowed: http://stackoverflow.com/a/1362192/1712389
If I move the JS to <head/> I get correct line numbers.
<head/>
Originally reported to the Atom linter-jshint repo (AtomLinter/linter-jshint#187), who directed me here.
The text was updated successfully, but these errors were encountered:
[[FIX]] (cli - extract) lines can end with "\\r\\n", not "\\n\\r"
a376c5b
Fixes jshintgh-2825
93818f3
096faa4
Successfully merging a pull request may close this issue.
Using Atom's linter-jshint, I'm getting JSHint errors on the wrong lines for inline JavaScript in the HTML body:
HTML source: https://gist.github.com/rmzelle/e5bc6f1a30113f69db9c
This is with Atom 1.3.2, linter 1.11.3, and linter-jshint 1.2.2 (with "Lint Inline Java Script" checked).
The HTML validates according to https://validator.w3.org/ and it looks like embedding JS in the body is indeed allowed: http://stackoverflow.com/a/1362192/1712389
If I move the JS to
<head/>
I get correct line numbers.Originally reported to the Atom linter-jshint repo (AtomLinter/linter-jshint#187), who directed me here.
The text was updated successfully, but these errors were encountered: