-
-
Notifications
You must be signed in to change notification settings - Fork 680
New: html-closing-bracket-spacing rule (fixes #229) #312
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 8000 a> and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two minor suggesions. Good job @mysticatea 👍
|
||
- :wrench: The `--fix` option on the [command line](http://eslint.org/docs/user-guide/command-line-interface#fix) can automatically fix some of the problems reported by this rule. | ||
|
||
This rule enforces space style before `>` of tags. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This rule enforces consistent spacing style before closing brackets
> of elements.
< 8000 /td> | { | |
code: '<template>\n <div >\n </div >\n <div/>\n</template>', | ||
errors: [ | ||
{ message: "Expected no space before '>', but found.", line: 2 }, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It would be also good to check whether column of the >
is properly reported.
Brilliant 🚀 |
Fixes #229.
This PR adds
vue/html-closing-bracket-spacing
rule. This is the spacing style version of #190. It requires or disallows a space before tag's closing brackets.