-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
docs: mark asyncData.pending as deprecated
#32959
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
Conversation
|
|
WalkthroughA deprecation JSDoc was added above the Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 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. 📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
⏰ 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). (19)
✨ 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/nuxt/src/app/composables/asyncData.ts (1)
119-122: Enrich the deprecation JSDoc with migration guidance and a reference linkInclude a short usage hint and a @see to guide users. This improves DX without changing behaviour.
/** - * @deprecated Use `status` instead. `pending` will be removed in Nuxt 5. + * @deprecated Use `status` instead (e.g. `status.value === 'pending'`). This property will be removed in Nuxt 5. + * @see https://nuxt.com/docs/api/composables/use-async-data#status */
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
packages/nuxt/src/app/composables/asyncData.ts(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{ts,tsx}
📄 CodeRabbit Inference Engine (.github/copilot-instructions.md)
Follow standard TypeScript conventions and best practices
Files:
packages/nuxt/src/app/composables/asyncData.ts
⏰ 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: build
- GitHub Check: codeql (javascript-typescript)
- GitHub Check: code
🔇 Additional comments (2)
packages/nuxt/src/app/composables/asyncData.ts (2)
119-122: Deprecation notice is clear and non-breakingThe JSDoc succinctly communicates the deprecation and the recommended alternative, with no runtime or type signature changes. This is safe to ship.
119-122: Confirm docs and migration notes are updated to reflect deprecationSince users often rely on
pending, please ensure the docs foruseAsyncDataexplicitly show the replacement pattern (status.value === 'pending') and call out the Nuxt 5 removal, so consumers aren’t caught off guard.
@nuxt/kit
nuxt
@nuxt/rspack-builder
@nuxt/schema
@nuxt/vite-builder
@nuxt/webpack-builder
commit: |
CodSpeed Performance ReportMerging #32959 will not alter performanceComparing Summary
|
huang-julien
left a comment
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.
I think we've decided to keep it, even for V5 🤔
|
Sure. I have added #32960 as place to discuss this :) |
fixes #32956
🔗 Linked issue
📚 Description
This PR marks
asyncData.pendingas deprecated and to be removed with v5.