Update database provider note in NestJS guide#7440
Update database provider note in NestJS guide#7440gideonadeti wants to merge 1 commit intoprisma:mainfrom
Conversation
WalkthroughA single documentation file was modified to remove PostgreSQL from the list of alternative driver adapters mentioned in an info note within the NestJS guide. The list now references only MySQL and SQL Server as alternative options for non-default providers. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
🚥 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 |
|
👋 Hi @gideonadeti! Thanks for your contribution to the Prisma docs. We wanted to let you know that we've recently updated our documentation structure and guidelines. Your PR may need to be updated to align with these changes. Action needed:
|
In the NestJS guide (section 2.1), the info block mentions PostgreSQL as an alternative database option. Since this specific guide is already built using PostgreSQL, listing it as an "alternative" is redundant and confusing for beginners. I have updated the text to clarify that other databases are supported, without listing the currently used database as an option.
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.