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(client, cli): client auto-install for pnpm #18646
Conversation
CodSpeed Performance ReportMerging #18646 Summary
Benchmarks breakdown
|
...-generators/generatorResolvers/prisma-client-js/check-dependencies/checkTypeScriptVersion.ts
Outdated
Show resolved
Hide resolved
...rc/get-generators/generatorResolvers/prisma-client-js/check-dependencies/checkYarnVersion.ts
Outdated
Show resolved
Hide resolved
...ternals/src/get-generators/generatorResolvers/prisma-client-js/check-dependencies/resolve.ts
Outdated
Show resolved
Hide resolved
...ternals/src/get-generators/generatorResolvers/prisma-client-js/check-dependencies/resolve.ts
Outdated
Show resolved
Hide resolved
...ternals/src/get-generators/generatorResolvers/prisma-client-js/check-dependencies/resolve.ts
Outdated
Show resolved
Hide resolved
...ges/internals/src/get-generators/generatorResolvers/prisma-client-js/prismaClientResolver.ts
Outdated
Show resolved
Hide resolved
...ges/internals/src/get-generators/generatorResolvers/prisma-client-js/prismaClientResolver.ts
Show resolved
Hide resolved
New dependency changes detected. Learn more about Socket for GitHub ↗︎ 👍 No new dependency issues detected in pull request Bot CommandsTo ignore an alert, reply with a comment starting with Ignoring: Pull request alert summary
📊 Modified Dependency Overview:
|
@SocketSecurity ignore |
packages/internals/src/get-generators/generatorResolvers/generatorResolvers.ts
Outdated
Show resolved
Hide resolved
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.
Left a question and pointed out one potential problem, otherwise LGTM
...ernals/src/get-generators/generatorResolvers/prisma-client-js/check-dependencies/semverLt.ts
Show resolved
Hide resolved
...ges/internals/src/get-generators/generatorResolvers/prisma-client-js/prismaClientResolver.ts
Show resolved
Hide resolved
packages/internals/src/get-generators/generatorResolvers/photonjs/photonResolver.ts
Outdated
Show resolved
Hide resolved
...-generators/generatorResolvers/prisma-client-js/check-dependencies/checkTypeScriptVersion.ts
Show resolved
Hide resolved
...rc/get-generators/generatorResolvers/prisma-client-js/check-dependencies/checkYarnVersion.ts
Show resolved
Hide resolved
...ges/internals/src/get-generators/generatorResolvers/prisma-client-js/prismaClientResolver.ts
Show resolved
Hide resolved
…use-ni-pm-detection
…use-ni-pm-detection
Fixes #5340
Fixes #9848
Fixes #14944
Fixes #11791
Fixes #14401
Fixes #14816
Fixes #18238
...likely to close yarn issues, but more tests are needed
Do not spend too much time on reviewing
@prisma/ni
, this package is temporary and is a port of my PR antfu-collective/ni#151 until it gets merged. You can review that PR too, if you find it useful.For the reviewer, please also review prisma/ecosystem-tests#3469