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 support for Reactivity Transform #1948

Closed
7 tasks
ota-meshi opened this issue Aug 17, 2022 · 5 comments
Closed
7 tasks

Add support for Reactivity Transform #1948

ota-meshi opened this issue Aug 17, 2022 · 5 comments

Comments

@ota-meshi
Copy link
Member

ota-meshi commented Aug 17, 2022

Lists the work this plugin needs to do to support Reactivity Transform.

Note
These changes will be released after the RFCs have been merged.

  • Related to new compiler macros.
  • Related to the change of defineProps().
    • Change vue/no-setup-props-destructure rule.
      Destructuring assignment immediately after defineProps() is allowed.
      However, Vue2 doesn't allow to use Reactivity Transform, so we may need to split rules.
    • Change vue/require-valid-default-prop rule.
    • Change vue/require-default-prop rule.
    • Change vue/no-boolean-default rule.
      We need to support a new way to specify defaults.

And more?

Additional context

Docs: https://vuejs.org/guide/extras/reactivity-transform.html
PR: vuejs/rfcs#420
Discussion: vuejs/rfcs#369

@ota-meshi
Copy link
Member Author

related to #1793, #1798, #1799, #1947

@sxzz
Copy link
Member

sxzz commented Mar 22, 2023

Since Reactivity Transform is dropped & deprecated, so can we just close this issue?

@ota-meshi
Copy link
Member Author

Thanks for letting me know! I will close this issue.

@ota-meshi ota-meshi unpinned this issue Apr 4, 2023
@bperel
Copy link

bperel commented May 24, 2023

Reactivity Transform has been moved to a Vue macro, would that make it reasonable to re-open this ticket, for developers who use this macro?

@sxzz
Copy link
Member

sxzz commented May 24, 2023

@bperel Vue Macros will add ESLint support soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants