fix(docs): clarify db push behavior in Prisma v7#7576
fix(docs): clarify db push behavior in Prisma v7#7576AmanVarshney01 merged 3 commits intoprisma:mainfrom
Conversation
Update documentation to reflect changes in Prisma v7 regarding db push and prisma generate.
|
@michael1-0 is attempting to deploy a commit to the Prisma Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughDocumentation update to Prisma migration workflow guide clarifying that in Prisma v7, Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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.
🧹 Nitpick comments (1)
apps/docs/content/docs/orm/prisma-migrate/workflows/prototyping-your-schema.mdx (1)
14-14: Use inline code formatting and explicit phrasing for command clarityOn Line 14, consider formatting commands as code and replacing “it” with the explicit command name to reduce ambiguity.
✏️ Suggested wording
-- In Prisma v7, db push no longer runs prisma generate automatically. Run it explicitly if needed. +- In Prisma v7, `db push` no longer runs `prisma generate` automatically. Run `prisma generate` explicitly if needed.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@apps/docs/content/docs/orm/prisma-migrate/workflows/prototyping-your-schema.mdx` at line 14, Change the sentence that mentions db push so commands are formatted as inline code and replace the vague pronoun "it" with the explicit command name; specifically, update the phrase "db push no longer runs prisma generate automatically. Run it explicitly if needed." to use backtick-style inline code for `db push` and `prisma generate` and replace "it" with "prisma generate" so the sentence reads that `db push` no longer runs `prisma generate` automatically and that you should run `prisma generate` explicitly when needed.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In
`@apps/docs/content/docs/orm/prisma-migrate/workflows/prototyping-your-schema.mdx`:
- Line 14: Change the sentence that mentions db push so commands are formatted
as inline code and replace the vague pronoun "it" with the explicit command
name; specifically, update the phrase "db push no longer runs prisma generate
automatically. Run it explicitly if needed." to use backtick-style inline code
for `db push` and `prisma generate` and replace "it" with "prisma generate" so
the sentence reads that `db push` no longer runs `prisma generate` automatically
and that you should run `prisma generate` explicitly when needed.
ℹ️ Review info
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
apps/docs/content/docs/orm/prisma-migrate/workflows/prototyping-your-schema.mdx
Remove redundant version reference since these are already v7 docs. Rephrase to match the style of surrounding bullet points.
|
Thanks for the PR @michael1-0 🫡 |
Update documentation to reflect changes in Prisma v7 regarding db push and prisma generate.
Summary by CodeRabbit
db pushno longer automatically runs generators. Users must now executeprisma generateexplicitly when needed.