Add new `vue/no-use-v-else-with-v-for` #2224

FloEdelmann · 2023-06-22T12:54:55Z

Fixes #1939.

ota-meshi

Thank you for this PR!
Almost looks good to me, but some unnecessary comments should be removed.

lib/rules/no-use-v-else-with-v-for.js

tests/lib/rules/no-use-v-else-with-v-for.js

FloEdelmann added 2 commits June 22, 2023 14:53

Improve vue/no-use-v-if-with-v-for description

b9b5186

Add new vue/no-use-v-else-with-v-for rule

72e3b2c

FloEdelmann requested a review from ota-meshi June 22, 2023 12:54

ota-meshi reviewed Jul 2, 2023

View reviewed changes

lib/rules/no-use-v-else-with-v-for.js Outdated Show resolved Hide resolved

tests/lib/rules/no-use-v-else-with-v-for.js Outdated Show resolved Hide resolved

Apply suggestions from code review

c5b53da

ota-meshi merged commit d1f1eea into master Jul 2, 2023
14 checks passed

ota-meshi deleted the no-use-v-else-with-v-for branch July 2, 2023 07:58

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add new `vue/no-use-v-else-with-v-for` #2224

Add new `vue/no-use-v-else-with-v-for` #2224

FloEdelmann commented Jun 22, 2023

ota-meshi left a comment

Add new vue/no-use-v-else-with-v-for #2224

Add new vue/no-use-v-else-with-v-for #2224

Conversation

FloEdelmann commented Jun 22, 2023

ota-meshi left a comment

Choose a reason for hiding this comment

Add new `vue/no-use-v-else-with-v-for` #2224

Add new `vue/no-use-v-else-with-v-for` #2224