-
-
Notifications
You must be signed in to change notification settings - Fork 89
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
fix: allow empty expr for dbgenerated() (#1400) #1401
Conversation
WalkthroughWalkthroughThe recent update modifies the Changes
Possibly related issues
Recent Review DetailsConfiguration used: CodeRabbit UI Files selected for processing (1)
Additional comments not posted (1)
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
Thank you for the fix @clementoriol ! Yes, Prisma's doc is a bit fuzzy about function details. I like that we can tweak to provide better flexibility. The change looks good to me and I'll merge it after CI passes. Will include it into a new patch release soon. |
Hey !
The fix for #1400 seemed easy enough (I mean who get the chance to submit a one char PR ? 😄) so here it goes.
The Prisma docs are not very clear on the subject but dbgenerated() can also accept no arguments for the use-case specified in the related ticket. (prisma docs: https://www.prisma.io/docs/orm/reference/prisma-schema-reference#dbgenerated)
However it does NOT supports empty strings but I don't think this can be expressed here.
Looking forward to any feedback.