-
Notifications
You must be signed in to change notification settings - Fork 6
Generate prisma client #65
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
| @@ -0,0 +1 @@ | |||
| export * from "./index" No newline at end of file | |||
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.
[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can prevent potential issues with some tools and version control systems.
| /* !!! This is code generated by Prisma. Do not edit directly. !!! | ||
| /* eslint-disable */ | ||
| // biome-ignore-all lint: generated file | ||
| module.exports = { ...require('.') } No newline at end of file |
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.
[maintainability]
Using require('.') without specifying a specific file or module can lead to unexpected behavior if the directory structure changes. Consider explicitly requiring the intended module for better maintainability and clarity.
| @@ -0,0 +1 @@ | |||
| export * from "./index" No newline at end of file | |||
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.
[💡 style]
Consider adding a newline at the end of the file to comply with POSIX standards and improve compatibility with various tools that expect a newline at the end of files.
| /* !!! This is code generated by Prisma. Do not edit directly. !!! | ||
| /* eslint-disable */ | ||
| // biome-ignore-all lint: generated file | ||
| module.exports = { ...require('#main-entry-point') } No newline at end of file |
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.
[❗❗ correctness]
The use of require('#main-entry-point') with a spread operator assumes that the module exports an object. If #main-entry-point exports something other than an object (e.g., a function or a primitive), this will cause a runtime error. Consider adding validation or handling for non-object exports to prevent potential issues.
| @@ -0,0 +1 @@ | |||
| export * from "./default" No newline at end of file | |||
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.
[💡 style]
Consider adding a newline at the end of the file. While this is not a correctness issue, it is a common convention that can help avoid unnecessary diffs in version control systems when lines are added to the end of the file.
|
|
||
| generator client { | ||
| provider = "prisma-client-js" | ||
| previewFeatures = ["fullTextSearchPostgres", "postgresqlExtensions", "views"] |
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.
[maintainability]
The previewFeatures option includes fullTextSearchPostgres, postgresqlExtensions, and views. Ensure these features are stable and necessary for your use case, as they might introduce breaking changes or unexpected behavior in future Prisma releases.
| generator externalClient { | ||
| provider = "prisma-client-js" | ||
| output = "../packages/challenge-prisma-client" | ||
| previewFeatures = ["fullTextSearchPostgres", "postgresqlExtensions", "views"] |
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.
[correctness]
The binaryTargets option specifies native and debian-openssl-3.0.x. Verify that these targets cover all environments where the Prisma client will be deployed, as missing targets could lead to runtime errors.
| @@ -8,6 +8,13 @@ generator client { | |||
| previewFeatures = ["fullTextSearchPostgres", "postgresqlExtensions", "views"] | |||
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.
[maintainability]
Consider whether all the preview features listed are necessary for your use case. Enabling unnecessary preview features can lead to unexpected behavior or instability as they are not yet fully supported.
| provider = "prisma-client-js" | ||
| output = "../packages/challenge-prisma-client" | ||
| previewFeatures = ["fullTextSearchPostgres", "postgresqlExtensions", "views"] | ||
| binaryTargets = ["native", "debian-openssl-3.0.x"] |
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.
[correctness]
The binaryTargets configuration has been moved from externalClient to client. Ensure that this change aligns with the intended deployment environments for the Prisma client. If externalClient requires specific binary targets, they should be explicitly defined.
Generated prisma client for external use