Skip to content

ci: policy 1.6, shared workflows v3.8.0 and the PR workflow - #31

Merged
boeboe merged 1 commit into
mainfrom
chore/policy-1.6-pr-ci
Nov 14, 2026
Merged

boeboe merged 1 commit into
mainfrom
chore/policy-1.6-pr-ci

Conversation

@boeboe

@boeboe boeboe commented Nov 14, 2026

Copy link
Copy Markdown
Contributor

Moves profinetctl to otfabric/engineering policy 1.6 and the current shared workflows (otfabric/.github v3.8.0), and to the PR workflow: heavy validation runs once per pull request revision, behind one required check.

What changes

  • pr.yml (new) runs on every pull request, with no path filter: docs always; ci unless every changed file is documentation (pr-changes.yml); then OT Fabric Required Checks (actions/required-gate), which fails closed.
  • Heavy workflows lose their push and pull_request triggers and become workflow_call + workflow_dispatch. Their jobs, matrices and permissions are unchanged. A schedule trigger stays.
  • No CI on a branch push or on the push to main any more, except the light checks on main. The merged commit was tested as the pull request's merge commit; a release tests the release commit again.
  • Policy pin 1.4 → 1.6 (.otfabric/policy.yaml); the checker runs at v1.6.1.
  • Pins: docs-check, go-binary-release, go-ci, pr-changes, required-gate at v3.8.0 (were v3.3.1, v3.5.0, v3.6.0). No input of a shared workflow was removed between those tags.
  • otfabric-policy runs once, in the light check (every pull request and every push to main), no longer a second time inside go-ci.

Files

  • .github/workflows/ci.yml
  • .github/workflows/docs.yml
  • .github/workflows/pr.yml (new)
  • .github/workflows/release.yml
  • .otfabric/policy.yaml
  • AGENTS.md
  • CONTRIBUTING.md

Quality gating

Every job that ran on a pull request before still runs on a pull request that changes code, and the single required check covers all of them. Nothing is weakened: no test, threshold, matrix entry or tool version is changed by this pull request.

Checks run locally

otfabric-policy check . at v1.6.1: compliant. make docs-check (docsync v0.3.1): clean. go build ./... and go vet ./...: clean.

Not run: actionlint.

🤖 Generated with Claude Code

Co-Authored-By: Claude Fable 5.2 <noreply@anthropic.com>
@codecov

codecov Bot commented Nov 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@boeboe
boeboe merged commit 2002781 into main Nov 14, 2026
17 checks passed
@boeboe
boeboe deleted the chore/policy-1.6-pr-ci branch November 14, 2026 03:49
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