feat: add no-deprecated-v-t rule
#580
Conversation
🦋 Changeset detectedLatest commit: 7e22f02 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Deploying eslint-plugin-vue-i18n with
|
| Latest commit: |
7e22f02
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://4efd1375.eslint-plugin-vue-i18n.pages.dev |
| Branch Preview URL: | https://feat-no-deprecated-v-t.eslint-plugin-vue-i18n.pages.dev |
docs/rules/no-deprecated-v-t.md
Outdated
| --- | ||
| title: '@intlify/vue-i18n/no-deprecated-v-t' | ||
| description: disallow using deprecated `v-t` custom directive (Deprecated in Vue I18n 11.0.0, removed fully in Vue I18n 12.0.0) | ||
| since: v3.2.0 |
There was a problem hiding this comment.
This should be added automatically upon release.
| since: v3.2.0 |
lib/rules/no-deprecated-v-t.ts
Outdated
| 'disallow using deprecated `v-t` custom directive (Deprecated in Vue I18n 11.0.0, removed fully in Vue I18n 12.0.0)', | ||
| category: 'Recommended', | ||
| url: 'https://eslint-plugin-vue-i18n.intlify.dev/rules/no-deprecated-v-t.html', | ||
| recommended: true |
There was a problem hiding this comment.
Changing shareable configuration may be considered a breaking change.
| recommended: true | |
| recommended: false |
I think that in the next major version we can clarify the versioning policy and allow configuration to change in minor versions. v10 of eslint-plugin-vue will do that too.
There was a problem hiding this comment.
I think that in the next major version we can clarify the versioning policy and allow configuration to change in minor versions. v10 of eslint-plugin-vue will do that too.
I agree too.
eslint-plugin-vue-i18n have not had versioning policy so far.
We should define it.
tests/lib/rules/no-deprecated-v-t.ts
Outdated
| }) | ||
|
|
||
| tester.run('no-deprecated-v-t', rule as never, { | ||
| valid: [], |
There was a problem hiding this comment.
Just to be sure, could you add a test case to verify that other directives are not detected?
|
@ota-meshi |
background
In vue-i18n v11,
v-tcustom directive is going to deprecated.vue-i18n has already published docs for breaking changes.
https://vue-i18n.intlify.dev/guide/migration/breaking11.html#deprecate-custom-directive-v-t
This rule supports migration.