docs(staging): de-Fly the staging host guidance#516
Conversation
Fly.io was abandoned for Railway (production runs on Railway). Replace the Fly.io `syd` recommendation with a host-agnostic note that names Railway as the production host and keeps Cloud Run as a Sydney-region alternative, and drop the now-false "Railway is unsuitable" line. Also generalise the operator-only account step from Fly.io to the host in use. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughUpdated staging and production deployment documentation to recommend any OCI-image host, identify Railway for production and runtime-secret examples, and retain Google Cloud Run in Sydney as an alternative. ChangesHosting Guidance
Estimated code review effort: 1 (Trivial) | ~2 minutes ✅ Pre-merge checks override appliedThe pre-merge checks have been overridden successfully. You can now proceed with the merge. Overridden by ❌ Failed checks (1 error)
✅ Passed checks (10 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 307893e6bd
ℹ️ 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".
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/staging-setup.md`:
- Around line 54-56: Synchronize the hosting recommendations in staging-setup.md
and deployment-architecture.md by selecting one authoritative production host.
Update both runbooks to name that host consistently, remove contradictory
suitability guidance, and preserve the existing regional alternative and
operator-step context where applicable.
🪄 Autofix (Beta)
✅ Autofix completed
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 87e96e8a-8568-4858-8e91-5462a3ba4903
📒 Files selected for processing (1)
docs/staging-setup.md
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. Fixes Applied SuccessfullyFixed 1 file(s) based on 1 unresolved review comment. Files modified:
Commit: The changes have been pushed to the Time taken: |
Fixed 1 file(s) based on 1 unresolved review comment. Co-authored-by: CodeRabbit <noreply@coderabbit.ai>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4d39fe85f1
ℹ️ 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".
What & why
Fly.io was abandoned for Railway (production runs on Railway). This purges the last Fly-specific guidance from
docs/staging-setup.md:**Fly.io \syd`**host recommendation with a host-agnostic note that names Railway as the production host and keeps Cloud Runaustralia-southeast2` as a Sydney-region alternative.Railway is unsuitable — no Sydney regionline (production is on Railway).Scope is intentionally limited to
docs/staging-setup.md. Not touched:.github/workflows/docker-image.yml— a general Docker image-build validator (used regardless of host); only mentions Fly in a comment.docs/deployment-architecture.md— has active uncommitted edits in another working branch; left for that work.The untracked
fly.toml+tests/fly-config.test.tsscratch files were removed from the working tree separately (they were never git-tracked, so nothing to PR there).Verification
fly/Flymentions remain indocs/staging-setup.md.prettier --checkclean.🤖 Generated with Claude Code