Skip to content

Moved Ghost update's analytics check into release qualification (S7) - #414

Merged
acburdine merged 1 commit into
next-dockerfrom
claude/roadmap-s7-into-s12
Oct 20, 2026
Merged

acburdine merged 1 commit into
next-dockerfrom
claude/roadmap-s7-into-s12

Conversation

@acburdine

Copy link
Copy Markdown
Member

ref https://linear.app/ghost/issue/PLA-485/s7-host-driven-ghost-upgrades-ghost-docker-update

S7's only remaining item was a hand-run acceptance check: on a site with analytics, update reruns Tinybird sync from the new image and the deploy succeeds. It needs a Tinybird login, so it cannot run in CI.

That check now sits with S12's other hand-run qualification scenarios, before the first stable release (PLA-483). With nothing left, S7's section and its entry in the delivery table are removed.

Documentation only.

🤖 Generated with Claude Code

ref https://linear.app/ghost/issue/PLA-485/s7-host-driven-ghost-upgrades-ghost-docker-update

S7 shipped except one acceptance check: on a site with analytics,
`update` reruns Tinybird sync from the new image and the deploy
succeeds. It needs a Tinybird login, so it cannot run in CI, and it
belongs with the other hand-run scenarios in S12's qualification. With
it there, S7 has nothing left, and its section leaves the roadmap.
@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 2554bc1d-4bb9-4e05-b8d3-aa7251d2b9ed
📥 Commits

Reviewing files that changed from the base of the PR and between c8e8ad8 and d59b119.

📒 Files selected for processing: 1
  • docs/ghost-cli-replacement.md

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Walkthrough

The documentation removes the S7 host-driven upgrade section and removes S7 analytics acceptance from the Admin-driven upgrade dependency. S12 qualification now requires an analytics-enabled update to rerun Tinybird sync from the new image and complete deployment. Manual execution requires a Tinybird login.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to d59b1

No actionable merge risk remains.

Pre-merge checks: Passed 5
Check name Status Explanation
Description check Passed The description accurately explains that the S7 analytics acceptance check moved into S12 release qualification and that the S7 documentation entries were removed.
Title check Passed The title clearly summarizes the main change: moving Ghost update's analytics check into release qualification under S7.
Docstring Coverage Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@acburdine
acburdine enabled auto-merge (squash) October 20, 2026 14:48
@acburdine
acburdine merged commit 17ad4e4 into next-docker Oct 20, 2026
17 checks passed
@acburdine
acburdine deleted the claude/roadmap-s7-into-s12 branch October 20, 2026 14:57
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