matt/mailship
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Remove dead build-in-production.sh (anchor fork artifact) #26

Merged
matt merged 1 commit from mailship-xyg-remove-dead-build-in-production-sh-ancho-e90 into main 2026-09-18 16:37:07 +00:00
Conversation 0 Commits 1 Files changed 1 -10
hudson commented 2026-09-18 16:35:07 +00:00
Collaborator
Copy link

mailship-xyg

Delete build-in-production.sh — a dead script left over from the anchor/mailship fork. Zero references anywhere in the codebase.

Changes:

  • Remove build-in-production.sh (10 lines). Its two responsibilities — removing pnpm link overrides and running the build — are already handled by the Dockerfile (which runs remove-pnpm-overrides.js directly). Its --no-frozen-lockfile workaround was a Render CI quirk irrelevant to the Docker build path. Keeping the file around misleads about deployment.
  • Confirmed via grep that no package.json script, Dockerfile, CI config, or render.yaml references this file. All existing quality gates (type-check, lint, build, 25 unit tests) pass.
mailship-xyg Delete `build-in-production.sh` — a dead script left over from the anchor/mailship fork. Zero references anywhere in the codebase. **Changes:** - Remove `build-in-production.sh` (10 lines). Its two responsibilities — removing pnpm link overrides and running the build — are already handled by the Dockerfile (which runs `remove-pnpm-overrides.js` directly). Its `--no-frozen-lockfile` workaround was a Render CI quirk irrelevant to the Docker build path. Keeping the file around misleads about deployment. - Confirmed via `grep` that no `package.json` script, Dockerfile, CI config, or `render.yaml` references this file. All existing quality gates (type-check, lint, build, 25 unit tests) pass.
hudson added 1 commit 2026-09-18 16:35:08 +00:00
Remove dead build-in-production.sh (anchor fork artifact)
All checks were successful
CI / checks (pull_request) Successful in 35s
Details
77ce571d52 
This file was a leftover from the anchor/mailship fork. Its function
(removing pnpm overrides, building) is already handled by the Dockerfile
via remove-pnpm-overrides.js. The --no-frozen-lockfile workaround is a
Render CI quirk irrelevant to Docker builds.

Zero references anywhere in the codebase — confirmed via grep.
matt merged commit 98e1de2ba0 into main 2026-09-18 16:37:07 +00:00
matt deleted branch mailship-xyg-remove-dead-build-in-production-sh-ancho-e90 2026-09-18 16:37:10 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: matt/mailship#26
No description provided.
Delete branch "mailship-xyg-remove-dead-build-in-production-sh-ancho-e90"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?