Skip to content

Fix auto-deploy: SSH repo alias, pin master, trigger on master - #290

Merged
AydinHassan merged 1 commit into
masterfrom
deploy-fix-master
Jul 5, 2026
Merged

AydinHassan merged 1 commit into
masterfrom
deploy-fix-master

Conversation

@AydinHassan

Copy link
Copy Markdown
Member

Finishes the Capistrano -> PHP Deployer migration so pushes to master auto-deploy phpschool.io.

Changes

  • deploy.php repository -> git@github-phpschool:... — imp's deploy user now has a per-repo read-only deploy key (phpschool_deploy) + Host github-phpschool SSH alias, avoiding the box-wide IdentitiesOnly github.com key that was bound to another repo.
  • deploy.php set('branch', 'master') — the CI runner checks out a detached HEAD, so Deployer would otherwise fall back to the repo default branch online (670 commits stale). Pinning guarantees master is deployed.
  • deploy.yml trigger online -> master (+ concurrency group) — the workflow only exists on master and online has no deploy workflow, so the deploy never fired.

Server prep (done on imp)

  • /var/www/phpschool.io manual deploy converted to Deployer layout: current -> releases/1, .dep/latest_release=1, shared symlinks repointed. releases/1 verified serving 200 as a rollback.

🤖 Generated with Claude Code

https://claude.ai/code/session_01P66wwaJwSSyXxza8ymKito

- deploy.php repository -> git@github-phpschool (per-repo deploy key on imp,
  avoids the shared IdentitiesOnly github.com key collision)
- Pin branch=master so Deployer checks out master (CI runner is detached HEAD,
  which would otherwise fall back to the stale default `online` branch)
- deploy.yml now fires on push to master (was `online`, which has no deploy
  workflow so the deploy never ran)

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P66wwaJwSSyXxza8ymKito
@AydinHassan
AydinHassan merged commit 63aeb68 into master Jul 5, 2026
1 check passed
@AydinHassan
AydinHassan deleted the deploy-fix-master branch July 5, 2026 11:48
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