Skip to content
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

Add vue/first-attribute-linebreak rule #1587

Merged
merged 1 commit into from Oct 20, 2021
Merged

Conversation

ota-meshi
Copy link
Member

@ota-meshi ota-meshi commented Jul 30, 2021

This PR adds vue/first-attribute-linebreak rule and removes the allowFirstLine option of the vue/max-attributes-per-line rule.
Also, add new vue/first-attribute-linebreak rule to the strongly-recommended and vue3-strongly-recommended rulesets.

vue/first-attribute-linebreak rule aims to enforce a consistent location for the first attribute.

By using "vue/first-attribute-linebreak": ["error", { "singleline": "beside"}], you can force the attributes that can be described in one line to one line.
So close the PR of #1431.

close #1431

@ota-meshi ota-meshi force-pushed the first-attribute-linebreak branch from 3de0ae8 to 42e5c8f Compare Aug 3, 2021
@ota-meshi ota-meshi merged commit 14a17d4 into master Oct 20, 2021
10 checks passed
@ota-meshi ota-meshi deleted the first-attribute-linebreak branch Oct 20, 2021
skjnldsv added a commit to nextcloud/eslint-config that referenced this pull request Jan 12, 2022
vuejs/eslint-plugin-vue#1587

Signed-off-by: John Molakvoæ <skjnldsv@protonmail.com>
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.

None yet

1 participant