-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
feat(nuxt): do not serialize skipHydrate properties #2798
Conversation
08257f9
to
b90150f
Compare
b90150f
to
4746ff2
Compare
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.
Thanks!
✅ Deploy Preview for pinia-playground ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for pinia-official ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
commit: |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v2 #2798 +/- ##
==========================================
- Coverage 90.11% 89.06% -1.05%
==========================================
Files 14 14
Lines 1335 1335
Branches 220 217 -3
==========================================
- Hits 1203 1189 -14
- Misses 131 145 +14
Partials 1 1 ☔ View full report in Codecov by Sentry. |
v2.2.6 fixed bug where `shouldHydrate()` is not exported that breaks `pnpm build` (ref. vuejs/pinia#2798)
Fixes #2773
Let me know if anything else is needed.