Adopt the unified sc deploy workflow #22

Open

geusebio wants to merge 5 commits from claude/adopt-unified-sc-workflow into main

pull from: claude/adopt-unified-sc-workflow

merge into: Grey.ooo:main

Grey.ooo:main

Grey.ooo:renovate/lock-file-maintenance

Grey.ooo:renovate/symfony

Grey.ooo:renovate/doctrine

Grey.ooo:testbed/verify-down

Grey.ooo:feature/pr-environment

Grey.ooo:fix/app-dir-traversable-permissions

Grey.ooo:claude/postgres-migration-pr4-2988a9

Grey.ooo:feature/deployment-per-branch

Grey.ooo:docs/sc-deploy-guide

Grey.ooo:claude/data-migration-doctrine-entity-bf52a5

Conversation 0 Commits 5 Files changed 4 +30 -76

geusebio commented 2026-08-25 23:14:05 +00:00

Owner

Copy link

Replaces pr-environment.yml, staging.yml and production.yml with a single deploy.yml that infers mode and deployment from the trigger instead of hardcoding it per file. Same three environments (PR previews, staging tracking main, production tracking the prod tag) plus branch-named deployments for anything else, all from one job.

See https://git.grey.ooo/Grey.ooo/someones.computer_pr-environment-action for the resolution rules and the deployment-report PR comment this now also posts.

Replaces pr-environment.yml, staging.yml and production.yml with a single deploy.yml that infers mode and deployment from the trigger instead of hardcoding it per file. Same three environments (PR previews, staging tracking main, production tracking the prod tag) plus branch-named deployments for anything else, all from one job. See https://git.grey.ooo/Grey.ooo/someones.computer_pr-environment-action for the resolution rules and the deployment-report PR comment this now also posts.

geusebio added 1 commit 2026-08-25 23:14:05 +00:00

Adopt the unified sc deploy workflow ... 4a80150dde

Replaces the three separate pr-environment/staging/production workflows
with one job that infers mode and deployment from whatever triggered it —
PR previews, a persistent staging deployment tracking main, a production
deployment tracking the prod tag, and one deployment per other branch, all
from the same step. See the pr-environment-action's own README/docs for the
resolution rules.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

geusebio added 1 commit 2026-08-25 23:22:53 +00:00

Pin the action to a commit until PR #1 merges upstream ... 4f5c7ada21

@main doesn't have the auto-detected mode/deployment logic yet — the push
that just landed on this branch hit the old resolve step's hard error
("no deployment name given ... pass 'deployment' explicitly"), and the
pull_request run silently deployed nothing (mode was never 'up' since
nothing sets it without the inference). Pinning to the commit that has the
fix until it merges.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

geusebio added 1 commit 2026-08-25 23:27:43 +00:00

Pass --endpoint explicitly — sc falls back to a local $SC_ENDPOINT here ... 589faa6ef4

The old workflows always set this explicitly; dropping it as "redundant
with the default" broke deploys on this org's runner image, which sets
$SC_ENDPOINT to a local test address that sc prefers over its own
https://someones.computer default when --endpoint isn't passed.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

geusebio added 1 commit 2026-08-25 23:33:23 +00:00

Bump the action back to @main now that PR #1 merged ... 47cf4bf3ed

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

geusebio added 1 commit 2026-08-25 23:39:59 +00:00

Retry: re-run deploy after platform-side streaming error 3b8a05d72f

All checks were successful

Hide checks

Deploy / deploy (pull_request) Successful in 10m10s

Details

Deploy / deploy (push) Successful in 10m10s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin claude/adopt-unified-sc-workflow:claude/adopt-unified-sc-workflow

git switch claude/adopt-unified-sc-workflow

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main

git merge --no-ff claude/adopt-unified-sc-workflow

git switch claude/adopt-unified-sc-workflow

git rebase main

git switch main

git merge --ff-only claude/adopt-unified-sc-workflow

git switch claude/adopt-unified-sc-workflow

git rebase main

git switch main

git merge --no-ff claude/adopt-unified-sc-workflow

git switch main

git merge --squash claude/adopt-unified-sc-workflow

git switch main

git merge --ff-only claude/adopt-unified-sc-workflow

git switch main

git merge claude/adopt-unified-sc-workflow

git push origin main

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

Subscribe

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

Grey.ooo/someones.computer_example_app!22

No description provided.