docs: fix missing comma in useMutation example to prevent syntax errors#9593
Conversation
WalkthroughA documentation snippet in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Poem
📜 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 as they are similar to previous changes (1)
✨ Finishing Touches🧪 Generate unit tests
🪧 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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
docs/framework/react/typescript.md (1)
252-255: Comma after spread correctly fixes the object literal syntax — LGTMWithout this comma, the snippet would throw a parsing error because
onSuccesswouldn’t be separated from the spread properties. The change is correct and improves copy-paste safety of the docs.For consistency with common Prettier settings on multi-line objects, consider adding a trailing comma after
onSuccessas well:useMutation({ ...groupMutationOptions(), - onSuccess: () => queryClient.invalidateQueries({ queryKey: ['groups'] }) + onSuccess: () => queryClient.invalidateQueries({ queryKey: ['groups'] }), })
📜 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)
docs/framework/react/typescript.md(1 hunks)
|
This LGTM |
|
View your CI Pipeline Execution ↗ for commit 999e531
☁️ Nx Cloud last updated this comment at |
Summary
Fixed a missing comma in the useMutation example's options object:
The comma after the spread operator was missing, which would cause a syntax error if users copy and paste the example.
Changes
This change maintains consistency with JavaScript object literal syntax rules and enhances developer experience by providing error-free examples.
Summary by CodeRabbit