-
Notifications
You must be signed in to change notification settings - Fork 867
Update Prisma schema path in overview document #7395
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 single documentation file was updated to modify a configuration example. The Prisma schema path in a multi-file schema usage snippet was changed from pointing to a specific file ( Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes This is a straightforward documentation example update affecting a single file. The change is minimal, localized, and cosmetic in nature—simply correcting the configuration path in an example snippet without introducing logic changes or new concepts. Possibly related PRs
Pre-merge checks✅ Passed checks (3 passed)
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. 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.
Actionable comments posted: 0
🧹 Nitpick comments (1)
content/200-orm/100-prisma-schema/10-overview/04-location.mdx (1)
72-72: Minor: Standardize path format across examples for consistency.The three examples use slightly different path formats for directory locations:
- Line 72:
./prisma(dot-slash prefix)- Line 78:
./prisma(dot-slash prefix)- Line 88:
prisma/(no dot-slash, trailing slash)All are valid, but standardizing the format would improve clarity. Consider updating line 88 to
./prismato match lines 72 and 78.export default defineConfig({ - schema: 'prisma/', + schema: './prisma',Also applies to: 78-78, 88-88
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
content/200-orm/100-prisma-schema/10-overview/04-location.mdx(1 hunks)
🔇 Additional comments (1)
content/200-orm/100-prisma-schema/10-overview/04-location.mdx (1)
88-88: ✓ Correctly updates schema path to directory format.The change aligns the
prisma.config.tsexample with the documented requirement (line 68) that multi-file Prisma schemas must use a directory path. This is now consistent with thepackage.jsonexample on line 78, and the warning section (line 107) properly reinforces this expectation.
|
Thanks for the PR @luzede ❤️ |
Changes schema path in the example to match a directory instead of a file. For multi file prisma schemas a directory needs to be provided that will have schema.prisma inside.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.