Skip to content

Enable vue/html-closing-bracket-* for top-level tags#1883

Merged
ota-meshi merged 3 commits intovuejs:masterfrom
FloEdelmann:top-level-html-closing-bracket
May 11, 2022
Merged

Enable vue/html-closing-bracket-* for top-level tags#1883
ota-meshi merged 3 commits intovuejs:masterfrom
FloEdelmann:top-level-html-closing-bracket

Conversation

@FloEdelmann
Copy link
Copy Markdown
Member

@FloEdelmann FloEdelmann commented May 7, 2022

Fixes #1879.

This should be released in the next major version since the bug fix could also be considered a breaking change.

Comment thread lib/rules/html-closing-bracket-newline.js Outdated
@FloEdelmann FloEdelmann requested a review from ota-meshi May 9, 2022 11:27
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 19c6e86 into vuejs:master May 11, 2022
@FloEdelmann FloEdelmann deleted the top-level-html-closing-bracket branch May 11, 2022 06:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

vue/html-closing-bracket-spacing (and more) does not work for top-level tags

2 participants