-
-
Notifications
You must be signed in to change notification settings - Fork 651
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
[Fix] Incorrect errors in invalid EOF cases (fixes #268) #274
Conversation
// code: '<template><div><input></input></div></template>', | ||
// output: '<template><div><input></template>', | ||
// errors: ["'<input>' should not have end tag."] | ||
// }, |
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.
Those tests are no longer needed. It's a kind of syntax error if void elements have their extra end tag, so it's reported by vue/no-parsing-error
rule.
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.
Okay, then please remove it :)
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 has been removed already 😀
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.
Apparently I need to start wearing glasses 🤓
// code: '<template><div><input></input></div></template>', | ||
// output: '<template><div><input></template>', | ||
// errors: ["'<input>' should not have end tag."] | ||
// }, |
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.
Okay, then please remove it :)
Please remove those commented lines and we're good to merge it. Good work! |
This PR fixes #268 and some similar cases:
I.e. some rules are confused in invalid EOF. This PR fixes some rules to not report errors if the code has invalid EOF.