Skip to content

ci: validate every example's compose files - #76

Merged
rickstaa merged 1 commit into
mainfrom
rs/ci-validate-compose
Aug 17, 2026
Merged

ci: validate every example's compose files#76
rickstaa merged 1 commit into
mainfrom
rs/ci-validate-compose

Conversation

@rickstaa

@rickstaa rickstaa commented Aug 17, 2026

Copy link
Copy Markdown
Member

Adds a compose job to the lint workflow that runs docker compose config over every example.

Every example pulls the orchestrator from ../compose.orchestrator.yml via extends, so one bad line there breaks all six at once and nobody runs all six before merging. Verified: a single ports: typo in that file fails all six, and this job catches it.

Builds and starts nothing, so it costs about a second per example. One env var, since api-proxy requires HF_TOKEN and everything else has a default.

Offchain only, deliberately: CI does not check the on-chain overlays.

Copilot AI lite review requested due to automatic review settings August 17, 2026 09:33

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI review requested due to automatic review settings August 17, 2026 10:14
@rickstaa
rickstaa force-pushed the rs/ci-validate-compose branch from 62fd4ad to df6956d Compare August 17, 2026 10:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

The shared orchestrator file feeds all six examples through `extends`, so
one bad line there breaks every one of them and nobody runs all six
before merging. Verified: a single typo in compose.orchestrator.yml fails
all six, and this catches it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01SD4o2Evku53Nb6zfgAdWxV
@rickstaa
rickstaa force-pushed the rs/ci-validate-compose branch from df6956d to b051c62 Compare August 17, 2026 10:17
Copilot AI review requested due to automatic review settings August 17, 2026 10:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@rickstaa
rickstaa merged commit dc24981 into main Aug 17, 2026
2 checks passed
@rickstaa
rickstaa deleted the rs/ci-validate-compose branch August 17, 2026 10:32
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.

2 participants