-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(get-platform): remove error, rephrase it as a warning #17389
feat(get-platform): remove error, rephrase it as a warning #17389
Conversation
…ithout custom compiled prisma engine
packages/get-platform/src/__tests__/parseOpenSSLVersion.test.ts
Outdated
Show resolved
Hide resolved
amd64
Alpine without custom compiled prisma engines
Integration PR: #17392 |
packages/get-platform/src/__tests__/parseOpenSSLVersion.test.ts
Outdated
Show resolved
Hide resolved
Decision points inspired from Slack:
const usesOfficialEngines = !(
process.env.PRISMA_QUERY_ENGINE_BINARY ||
process.env.PRISMA_QUERY_ENGINE_LIBRARY ||
process.env.PRISMA_MIGRATION_ENGINE_BINARY
)
|
…icial-prisma-engines
…a-engines' of github.com:prisma/prisma into feat/musl-not-amd64-only-fail-when-using-official-prisma-engines
amd64
Alpine without custom compiled prisma enginesThere 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.
@jkomyno Could you remove this binary packages/engines/introspection-engine-darwin-arm64
and force push?
Closes #17390.
@prisma/get-platform
TODO
https://pris.ly/d/docker-alpine
to the tutorial generated by (Serverless: OS Support Initiative): Document how to use Prisma with Docker / Linux Distros docs#4365