Skip to content

docs: mention Server mode + ci.forkhorizon.com production backend#81

Merged
Daliys merged 1 commit into
mainfrom
docs/readme-backend-clarify
Jul 25, 2026
Merged

docs: mention Server mode + ci.forkhorizon.com production backend#81
Daliys merged 1 commit into
mainfrom
docs/readme-backend-clarify

Conversation

@Daliys

@Daliys Daliys commented Jul 25, 2026

Copy link
Copy Markdown
Member

Why

main's README only documents the localhost-only webhook fallback path and never mentions Server mode — even though the broker code that implements it (CIQueueSettingsStore.swift, LocalBrokerLaunchAgent.swift) is present on main too. A reader of just this branch would have no idea the production setup (ci.forkhorizon.com, served by ForkHorizon/WebSite) exists.

What changed

  • Added a short paragraph introducing Server mode and naming the real production backend, pointing to develop's README for full setup detail rather than duplicating it (see the companion PR on develop, which also corrects a wrong claim about where that backend lives).

Left as a draft — not merging myself, surfacing for review.

main's README only documented the localhost webhook-only fallback path
and never mentioned that the broker also has a Server mode talking to a
real backend (ci.forkhorizon.com, served by ForkHorizon/WebSite). That
Server mode already exists in the broker code on both main and develop
(CIQueueSettingsStore.swift, LocalBrokerLaunchAgent.swift) even though
develop's README docs for it were pointing at the wrong backend (see
companion PR on develop). Points main's reader at develop for full setup
details rather than duplicating them here.
@Daliys
Daliys marked this pull request as ready for review July 25, 2026 18:59
@Daliys
Daliys merged commit 360bacc into main Jul 25, 2026
2 of 4 checks passed
@Daliys
Daliys deleted the docs/readme-backend-clarify branch July 25, 2026 18:59
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