Skip to content

vue/html-end-tags rule goes out of control #268

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.

Sign up for GitHub

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

Jump to bottom
Closed
mysticatea opened this issue Dec 1, 2017 · 3 comments · Fixed by #274
Closed

vue/html-end-tags rule goes out of control #268

mysticatea opened this issue Dec 1, 2017 · 3 comments · Fixed by #274
Assignees
@mysticatea
Labels
bug

Comments

@mysticatea
Copy link
Member

Tell us about your environment

  • ESLint Version: 4.12.1
  • eslint-plugin-vue Version: 4.0.0-beta.0

Please show your full configuration:
What did you do? Please include the actual source code causing the issue.

See playground.

What did you expect to happen?

Autofix does not change the code.

What actually happened? Please include the actual, raw output from ESLint.

The autofix of vue/html-end-tags added many </template> tags.
The rule should do nothing if an eof-in-tag error exists.
@mysticatea mysticatea added the bug label Dec 1, 2017
mysticatea added a commit that referenced this issue Dec 2, 2017
@mysticatea
Fix: incorrect errors in invalid EOF cases (fixes #268)
bc734e4
@mysticatea mysticatea mentioned this issue Dec 2, 2017
Merged
@mysticatea mysticatea self-assigned this Dec 2, 2017
michalsnik pushed a commit that referenced this issue Dec 3, 2017
@mysticatea @michalsnik
Fix: incorrect errors in invalid EOF cases (fixes #268) (#274)
a93a234
michalsnik pushed a commit that referenced this issue Dec 3, 2017
@mysticatea @michalsnik
[Update] Make vue/html-quotes fixable (#275)
7897252 
* Fix: incorrect errors in invalid EOF cases (fixes #268)

* Update: make vue/html-quotes fixable
@PLQin
Copy link

PLQin commented Jan 3, 2019

@michalsnik so, How is the problem solved?

@PLQin
Copy link

PLQin commented Jan 3, 2019

@michalsnik so, How is the problem solved?

What should I do ?

@michalsnik
Copy link
Member

This has been fixed one year ago, so if you use recent version of eslint-plugin-vue, this should no longer be the case. If you're having problems please create new issue with detailed description @PLQin

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mysticatea mysticatea

Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fix] Incorrect errors in invalid EOF cases (fixes #268) vuejs/eslint-plugin-vue
3 participants
@mysticatea @michalsnik @PLQin