Skip to content

Remove: postinstall script#3

Merged
Prajjawalk merged 2 commits intomainfrom
feat/backend-port
Mar 8, 2026
Merged

Remove: postinstall script#3
Prajjawalk merged 2 commits intomainfrom
feat/backend-port

Conversation

@Prajjawalk
Copy link
Copy Markdown
Contributor

@Prajjawalk Prajjawalk commented Mar 8, 2026

Summary by CodeRabbit

  • Chores
    • Streamlined build initialization by moving dependency generation to the continuous integration pipeline instead of local installation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 8, 2026

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

The Prisma code generation step was relocated from a postinstall hook in package.json to an explicit build step in the GitHub Actions CI workflow, running after the dependency installation step.

Changes

Cohort / File(s) Summary
CI/Workflow Configuration
.github/workflows/ci.yml
Added bunx prisma generate step immediately after bun install --frozen-lockfile.
Package Configuration
package.json
Removed the postinstall script entry that executed prisma generate.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 Prisma's code now generates in CI's gleaming light,
No more postinstall hooks in the dead of night,
The workflow's explicit, the build steps align,
Generation flows steady through the pipeline divine!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: removing the postinstall script from package.json, which aligns with the main modification in the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/backend-port

Comment @coderabbitai help to get the list of available commands and usage tips.

@Prajjawalk Prajjawalk merged commit 39c9a0e into main Mar 8, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant