Skip to content

chore(vue): Use dedicated vue/no-layout-rules config#69

Merged
lydell merged 2 commits intoprettier:masterfrom
michalsnik:update-eslint-plugin-vue
Jan 3, 2019
Merged

chore(vue): Use dedicated vue/no-layout-rules config#69
lydell merged 2 commits intoprettier:masterfrom
michalsnik:update-eslint-plugin-vue

Conversation

@michalsnik
Copy link
Copy Markdown
Contributor

This PR updates eslint-plugin-vue, as well as updates vue config, by using brand new vue/no-layout-rules config, that is automatically generated based on the type property in rules' metadata. This way we won't get out-of-sync :)

@lydell lydell merged commit 089a7b5 into prettier:master Jan 3, 2019
@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 3, 2019

Thanks!

@lydell
Copy link
Copy Markdown
Member

lydell commented Jan 13, 2019

Release in v3.4.0.

lydell added a commit that referenced this pull request Jan 16, 2019
Fixes #70. While the PR was well intended, it unfortunately was breaking
change since it requires eslint-plugin-vue@5 while lots of people still
use version 4.

Also, sometimes there are layout rules that don't conflict with Prettier
but rather complement it. So it might make sense to keep a manual,
Prettier-specific list after all.
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.

3 participants