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

vue: Update eslint-plugin-vue to 7.4.1 #348

Merged
merged 2 commits into from
Jan 20, 2021
Merged

vue: Update eslint-plugin-vue to 7.4.1 #348

merged 2 commits into from
Jan 20, 2021

Conversation

edg2s
Copy link
Member

@edg2s edg2s commented Jan 6, 2021

No description provided.

@edg2s
Copy link
Member Author

edg2s commented Jan 6, 2021

Current failures:

/var/www/eslint-config-wikimedia/test/fixtures/vue-common/invalid.vue
  11:14  warning  Expected '#name' instead of 'v-slot:name'                                                     vue/v-slot-style
  15:14  warning  Expected '#name' instead of 'v-slot:name'                                                     vue/v-slot-style
  15:14  error    An element cannot have multiple '<template>' elements which are distributed to the same slot  vue/valid-v-slot
  34:2   warning  Unused eslint-disable directive (no problems were reported from 'vue/name-property-casing')
  35:8   warning  Property name "div" is not PascalCase                                                         vue/component-definition-name-casing

✖ 5 problems (1 error, 4 warnings)
  0 errors and 3 warnings potentially fixable with the `--fix` option.

cc @catrope

@jdforrester
Copy link
Member

(Waiting for final "go" from Roan.)

@jdforrester
Copy link
Member

Also, once this lands, we'll cut a new release, I think; see #340.

@catrope catrope changed the title vue: Update eslint-pluing-vue to 7.4.1 vue: Update eslint-plugin-vue to 7.4.1 Jan 20, 2021
@catrope catrope merged commit f7f6178 into master Jan 20, 2021
@edg2s edg2s deleted the vue branch January 20, 2021 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants