-
-
Notifications
You must be signed in to change notification settings - Fork 1k
Conversation
✔️ Deploy Preview for nuxt3-docs canceled. 🔨 Explore the source changes: 9de1877 🔍 Inspect the deploy log: https://app.netlify.com/sites/nuxt3-docs/deploys/6233b24e367d2300099af85b |
Co-authored-by: Anthony Fu <anthonyfu117@hotmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Shall we also add a basic test for this flag?
Since it's kinda global - do you want it to be enabled for the basic fixture or create a new fixture for it? |
It would be great to test with same fixture if possible to save CI time 😊 |
Tests are still failing since |
@antfu nuxt bridge error, because not set
|
@baixiaoyu2997 Bridge does not support reactivityTransform currently, it's a Nuxt 3 only feature |
我没有开启experimental这个配置,但是报错了:
from this code:
如果需要复现 我可以提供在线代码 |
same problem: nuxt/bridge#84 |
Issues with bridge should be solved by #3763 |
Is it normal that the documentation has not been generated for the |
Does reactivity transform work with nitro? When importing $ref from 'vue/macros' my IDE seems to find the correct file (.../node_modules/.pnpm/vue@3.2.37/node_modules/vue/macros.d.ts) but Rollup is giving me an error, saying it could not load .../node_modules/.pnpm/vue@3.2.37/node_modules/vue/dist/vue.cjs.js/macros. I tried explicitly installing Vue 3, as suggested in https://github.com/nuxt/framework/discussions/2028, but still got the error. Reproduction: |
🔗 Linked issue
nuxt/nuxt#13105
❓ Type of change
📚 Description
Enable by
experimental.reactivityTransform: true
📝 Checklist