Skip to content

ci: gate merges on a single ci-gate check - #135

Merged
martinothamar merged 2 commits into
mainfrom
agentctl/prep
Oct 5, 2026
Merged

martinothamar merged 2 commits into
mainfrom
agentctl/prep

Conversation

@martinothamar

@martinothamar martinothamar commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Prepares the repository for moving agentctl and the Sandbox crates in from altinn-studio (#136, stacked on this).

  • CI: each area of the monorepo runs as a reusable workflow, only when its paths change, and ci-gate becomes the one required status check. It fails when a job that ran failed or was cancelled, and passes when areas were skipped, so new areas can be added without touching branch protection. integrations moves unchanged into integrations.yml. Path detection is a short shell step over the GitHub API's file list, and pushes to main are filtered the same way. Superseded PR runs are cancelled.
  • License: the copyright holder becomes The Norwegian Digitalisation Agency.
  • README.md and doc/pr-prosess.md name ci-gate as the required check.

Merging needs one branch protection change: inside the reusable workflow the job reports as integrations / integrations, so the currently required integrations check never arrives on this PR. Once ci-gate has run here, the required check moves from integrations to ci-gate, and then this merges.

Run each area of the monorepo as a reusable workflow that only runs
when its paths change, and make ci-gate the one required status check.
It fails when a job that ran failed or was cancelled, and passes when
areas were skipped, so new areas can be added without changing branch
protection. integrations moves unchanged into integrations.yml.
@martinothamar
martinothamar added this pull request to stack #137 October 5, 2026 07:16
@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ce7d7c32-0db4-4999-8c5e-763df0a1728e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@martinothamar martinothamar changed the title agentctl/prep ci: gate merges on a single ci-gate check Oct 5, 2026
@martinothamar
martinothamar marked this pull request as ready for review October 5, 2026 20:20
@martinothamar
martinothamar merged commit 858f5fb into main Oct 5, 2026
6 checks passed
@martinothamar
martinothamar deleted the agentctl/prep branch October 5, 2026 21:43
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