Support for defineModel
#2130
Comments
This comment was marked as resolved.
This comment was marked as resolved.
It is now stable in Vue 3.4: https://blog.vuejs.org/posts/vue-3-4#definemodel-is-now-stable |
|
Rules that need to be changed (feel free to edit and expand):
|
LeSuisse
pushed a commit
to Enalean/tuleap
that referenced
this issue
Jan 8, 2024
Vue 3.4 announcement: https://blog.vuejs.org/posts/vue-3-4 This allows us to use `defineModel` macro. This remove an eslint-disable but introduce a new one 😒 since eslint needs to be updated upstream to support defineModel. See following issue for details: vuejs/eslint-plugin-vue#2130 Jest doesn't know defineModel either, therefore we take the opportunity to switch to vitest. One bird two stones 😅 See documentation about defineModel: https://vuejs.org/guide/components/v-model.html No functional changes: in project » admin » permissions per group. Part of request #31168: Migration Vue2 => Vue3 Change-Id: Idbd1a1519460391cc4438a951865415745ff2dd5
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Tell us about your environment
The problem you want to solve.
Add support for
defineModelmacro.See https://github.com/vuejs/core/blob/main/CHANGELOG.md#330-alpha9-2023-04-08
Your take on the correct solution to problem.
Parser and some rules are affected. However, we have not yet listed the affected rules.
I think that, first, we need to list the rules affected by defineModel.
Additional context
Related to vuejs/vue-eslint-parser#186
The text was updated successfully, but these errors were encountered: