Skip to content

[codex] Add edge push deploy workflow#1

Merged
seslly merged 2 commits into
masterfrom
codex/edge-push-deploy
Jun 26, 2026
Merged

[codex] Add edge push deploy workflow#1
seslly merged 2 commits into
masterfrom
codex/edge-push-deploy

Conversation

@seslly

@seslly seslly commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds an app-owned GitHub Actions deploy path for the Shipstuff DigitalOcean edge.

Changes:

  • validates/builds the app on pull requests
  • uploads static assets or backend release artifacts with shipstuff/actions/digital-ocean-upload
  • uses the production environment EDGE_KEY secret that the servertimeai operator playbook creates

Validation

  • workflow YAML parsed locally
  • repo-specific smoke/build checks were run where practical before opening this PR

Note: production deploy remains blocked until the matching servertimeai operator playbook has been run for this service.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 510be62664

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml
pull_request:
push:
branches:
- master

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Point the deploy workflow at main

In this checkout the integration branch is main (git branch --all shows no master branch), but this trigger only runs for pushes to master; the deploy job also repeats refs/heads/master. If merges land on main, the new push validation and edge deploy path will not run at all for this repo, so production deploys remain silently disabled.

Useful? React with 👍 / 👎.

@seslly
seslly merged commit efa827f into master Jun 26, 2026
2 checks passed
@seslly
seslly deleted the codex/edge-push-deploy branch June 26, 2026 04:24
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