fix(cli): support env files in app deploy#72
Conversation
Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (10)
Summary by CodeRabbit
WalkthroughThis PR extends the 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
Signed-off-by: Aman Varshney <amanvarshney.work@gmail.com>
Summary
prisma-cli app deploy --env .envalongside--env NAME=VALUE--env <name=value|file>Why
app deploy --envcurrently only accepts inlineNAME=VALUEvalues. Passing.envis treated as an invalid assignment because the deploy path always calls the assignment parser directly, even though the CLI already has dotenv file parsing for env commands.Validation
pnpm --filter @prisma/cli testpnpm --filter @prisma/cli buildnode packages/cli/dist/cli.js app deploy --prod --http-port 8080 --env .env --yes --json