Skip to content

feat(no-v-text-v-html-on-component): add ignore namespace option#2610

Merged
ota-meshi merged 6 commits intovuejs:masterfrom
waynzh:feature/ignore-namespace
Nov 27, 2024
Merged

feat(no-v-text-v-html-on-component): add ignore namespace option#2610
ota-meshi merged 6 commits intovuejs:masterfrom
waynzh:feature/ignore-namespace

Conversation

@waynzh
Copy link
Copy Markdown
Member

@waynzh waynzh commented Nov 16, 2024

resolve #2458

Comment thread docs/rules/no-v-text-v-html-on-component.md Outdated
Comment thread docs/rules/no-v-text-v-html-on-component.md Outdated
Comment thread lib/rules/no-v-text-v-html-on-component.js Outdated
Comment thread tests/lib/rules/no-v-text-v-html-on-component.js
waynzh and others added 4 commits November 19, 2024 12:05
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
Copy link
Copy Markdown
Member

@FloEdelmann FloEdelmann left a comment

Choose a reason for hiding this comment

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

Looks good to me, thanks 🙂

@FloEdelmann
Copy link
Copy Markdown
Member

The test failure is probably caused by an ESLint update, so no need to worry about this.

Copy link
Copy Markdown
Member

@ota-meshi ota-meshi left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@ota-meshi ota-meshi merged commit bea53c0 into vuejs:master Nov 27, 2024
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.

Namespace handling makes it impossible to lint recursive SVG / MathML components

3 participants