Skip to content

ci: weekly macOS and Windows leg; docs: scope split; register release.yml - #1

Merged
imanimanyara merged 2 commits into
mainfrom
ci/weekly-cross-os
Nov 3, 2026
Merged

imanimanyara merged 2 commits into
mainfrom
ci/weekly-cross-os

Conversation

@imanimanyara

@imanimanyara imanimanyara commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Cross-OS CI. dbpantry and mesobia ship binaries for macOS and Windows, so cloudkit has to build and pass there. A cross-os job (macOS and Windows, make ci with the race detector) runs weekly and on dispatch; pull requests keep testing on Linux only. It was proven by one dispatch on this branch.
  • Scope in the docs. docs/architecture.md now states the split the owner settled: cloudkit finds and connects to managed databases but never administers them (that stays in mesobia's engine adapters), and REST descriptors, still planned, support writes behind signing or explicit enablement, plans against live state, typed destroy confirmation and the contract suite.
  • Registering release.yml. GitHub did not register the dispatch-only release workflow from the first push, so a dispatch would answer 404. Changing the file here means its merge registers it. docs/release.md gains a step-0 check.

This is also the repository's first CI run: the Initial release push to main started none, because CI triggers on pull_request, dispatch and the weekly schedule only.

Checks

  • make ci and golangci-lint pass locally
  • CHANGELOG.md has [Unreleased] entries

dbpantry and mesobia ship binaries for macOS and Windows, so cloudkit has to build and pass there. Pull requests keep testing on Linux only.
Copilot AI balanced review requested due to automatic review settings November 3, 2026 08:16

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

…se.yml

Managed-database administration stays with the consumer and descriptors support writes (mesobia ADR 0046 in the design record). release.yml changes so its merge registers the dispatch-only workflow, which the first push did not.
@imanimanyara imanimanyara changed the title ci: test on macOS and Windows weekly and on dispatch ci: weekly macOS and Windows leg; docs: scope split; register release.yml Nov 3, 2026
@imanimanyara
imanimanyara merged commit c1a6a98 into main Nov 3, 2026
12 checks passed
@imanimanyara
imanimanyara deleted the ci/weekly-cross-os branch November 3, 2026 12:54
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