-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Mention nuxt 4 in docs #3024
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
Mention nuxt 4 in docs #3024
Conversation
✅ Deploy Preview for pinia-official ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
WalkthroughUpdated documentation in packages/docs/ssr/nuxt.md to state that Pinia supports Nuxt Bridge, Nuxt 3, and Nuxt 4. No code changes. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
packages/docs/ssr/nuxt.md (1)
8-8
: Nit: tighten wording, hyphenate “server-side”, and use Oxford commaMinor style polish: avoid “a lot of”, prefer “server-side”, fix “nor” → “or”, and add the serial comma before “and Nuxt 4” (if consistent with house style).
Apply this diff:
-Using Pinia with [Nuxt](https://nuxt.com/) is easier since Nuxt takes care of a lot of things when it comes to _server side rendering_. For instance, **you don't need to care about serialization nor XSS attacks**. Pinia supports Nuxt Bridge, Nuxt 3 and Nuxt 4. For bare Nuxt 2 support, [see below](#nuxt-2-without-bridge). +Using Pinia with [Nuxt](https://nuxt.com/) is simpler because Nuxt takes care of many aspects of _server-side rendering_. For instance, **you don't need to care about serialization or XSS attacks**. Pinia supports Nuxt Bridge, Nuxt 3, and Nuxt 4. For bare Nuxt 2 support, [see below](#nuxt-2-without-bridge).
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
packages/docs/ssr/nuxt.md
(1 hunks)
🧰 Additional context used
🪛 LanguageTool
packages/docs/ssr/nuxt.md
[style] ~8-~8: Consider using a synonym to be more concise.
Context: ...om/) is easier since Nuxt takes care of a lot of things when it comes to _server side re...
(A_LOT_OF)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)
- GitHub Check: Redirect rules
- GitHub Check: Header rules
- GitHub Check: Pages changed
🔇 Additional comments (1)
packages/docs/ssr/nuxt.md (1)
8-8
: LGTM: Thanks for calling out Nuxt 4 support.Doc update is accurate and scoped. No functional impact.
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!
Mentioned Nuxt 4 in the docs, since its already supported.
Summary by CodeRabbit