-
-
Notifications
You must be signed in to change notification settings - Fork 679
valid-template-root should allow multiple tags if part of same v-if #884
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
Comments
Thank you for this issue, but I could not reproduce it: online demo Please use the latest version. |
i have the same issue as @rightaway
edit: the same code that triggers an error in my editor, works just fine with the vue-eslint-demo |
@andre-brdoch I experienced the same issue. However when I removed |
@andre-brdoch What do you think the issue is if it works in the demo but not in your editor? |
This should fix all medium-and-above severity security issues reported by npm audit. The only remaining issue that cannot be fixed by running `npm upgrade` or `npm audit fix` will be fixed in a follow-up commit. The rule to enforce a single root in vue templates seems to be a bit overzealous. This has already been reported upstream in vuejs/eslint-plugin-vue#971 and was also discussed in vuejs/eslint-plugin-vue#884 and vuejs/eslint-plugin-vue#986 without obvious solution that would be applicable here. Solved by adding a not strictly necessary wrapping span. Change-Id: I3153c0f45ce53704ef2c02d7e2e2e4f6d67e3fc7
* Update Wikibase from branch 'master' to e7ada5fde5f187fef50badd632676363f322691f - Merge "bridge: Upgrade dependencies" - bridge: Upgrade dependencies This should fix all medium-and-above severity security issues reported by npm audit. The only remaining issue that cannot be fixed by running `npm upgrade` or `npm audit fix` will be fixed in a follow-up commit. The rule to enforce a single root in vue templates seems to be a bit overzealous. This has already been reported upstream in vuejs/eslint-plugin-vue#971 and was also discussed in vuejs/eslint-plugin-vue#884 and vuejs/eslint-plugin-vue#986 without obvious solution that would be applicable here. Solved by adding a not strictly necessary wrapping span. Change-Id: I3153c0f45ce53704ef2c02d7e2e2e4f6d67e3fc7
valid-template-root
gives an error if you have the following but it is allowed by Vue so this shouldn't give an error. Same applies if you have 1 or more v-else-if elements there.The text was updated successfully, but these errors were encountered: