Skip to content

Enable ESLint rules vue/valid-v-for and vue/require-v-for-key#399

Merged
brianhelba merged 2 commits into
masterfrom
vue-v-for
Jun 5, 2026
Merged

Enable ESLint rules vue/valid-v-for and vue/require-v-for-key#399
brianhelba merged 2 commits into
masterfrom
vue-v-for

Conversation

@brianhelba
Copy link
Copy Markdown
Collaborator

No description provided.

@brianhelba brianhelba requested a review from annehaley June 5, 2026 18:21
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying geodatalytics with  Cloudflare Pages  Cloudflare Pages

Latest commit: af430a9
Status: ✅  Deploy successful!
Preview URL: https://6a36c7e7.geodatalytics.pages.dev
Branch Preview URL: https://vue-v-for.geodatalytics.pages.dev

View logs

Comment thread web/eslint.config.ts
Comment on lines -23 to -25
// Temporary ignores until rules can be fixed
"vue/require-v-for-key": "off",
"vue/valid-v-for": "off",
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, looks like these were the last ones that needed to be addressed. Tysm for doing all that cleanup!

@brianhelba brianhelba merged commit 8dc0534 into master Jun 5, 2026
4 checks passed
@brianhelba brianhelba deleted the vue-v-for branch June 5, 2026 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants