Skip to content

ci: add PR validation with canonical ok check#6

Merged
CameronBrooks11 merged 2 commits into
mainfrom
ci/add-ok-check
Jun 18, 2026
Merged

ci: add PR validation with canonical ok check#6
CameronBrooks11 merged 2 commits into
mainfrom
ci/add-ok-check

Conversation

@CameronBrooks11

Copy link
Copy Markdown
Member

Onboards baseliner-control into the org branch-protection convention.

This repo previously had no PR-triggered CIbaseliner.yml only runs on a weekly schedule + manual dispatch — so it could not carry the canonical ok required status check.

This adds ci.yml running on pull_request with:

  • validatebaseliner policy --config baseliner.yaml (resolves the policy offline, no token; fails on a malformed/invalid config) + a JSON well-formedness check on renovate.json
  • ok — the standard aggregator job every repo exposes

Once green and merged, the repo gets added to REPOS in anolishq/.githubscripts/apply-branch-protection.sh and protected like the rest.

Adds a CI workflow that runs on pull_request so this repo exposes the
org-wide `ok` required status check. The validate job resolves
baseliner.yaml via `baseliner policy` (offline, no token) and checks
renovate.json is well-formed JSON.

Prereq for onboarding into apply-branch-protection.sh.
The previous run value started with a double-quote, which YAML parses as
a quoted scalar with trailing content — an invalid workflow that failed
to compile (run produced zero jobs). Use a block scalar like the scan
workflow does.
@CameronBrooks11
CameronBrooks11 merged commit b0ca1b8 into main Jun 18, 2026
2 checks passed
@CameronBrooks11
CameronBrooks11 deleted the ci/add-ok-check branch June 18, 2026 02:35
CameronBrooks11 added a commit to anolishq/.github that referenced this pull request Jun 18, 2026
Adds baseliner-control to the REPOS list now that its CI exposes the
shared `ok` aggregator check (anolishq/baseliner-control#6). Brings the
canonical classic main protection to all 14 org repos.
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