Skip to content

ci(fork): run Validate lime-elements on PRs targeting lime#7

Merged
adrianschmidt merged 1 commit into
limefrom
ci/validate-lime-elements-on-fork
Jun 17, 2026
Merged

ci(fork): run Validate lime-elements on PRs targeting lime#7
adrianschmidt merged 1 commit into
limefrom
ci/validate-lime-elements-on-fork

Conversation

@adrianschmidt

Copy link
Copy Markdown

What

Add lime to the base-branch filter of the Validate lime-elements workflow trigger:

 on:
   pull_request:
-    branches: [main]
+    branches: [lime, main]

Why

The workflow came over from upstream jgroth/kompendium and only triggered on PRs to main. But this fork's default branch is lime, and all fork PRs target lime — so the check never ran (PR #6 shows only Block Autosquash / CI / Lint; the workflow has zero runs).

For pull_request, the branches filter matches the PR's base branch, so adding lime is what makes it fire on our PRs. main is kept for any main-targeted PRs and to minimize divergence from upstream.

Nothing else needed adapting: the workflow already checks out the public Lundalogik/lime-elements@main, and the build/link steps are repo-agnostic.

Self-test

This PR's base is lime and its head contains the new trigger, so the Validate lime-elements check should run on this PR itself.

Note

This is a fork customization on a file that sync-upstream.yml pulls from upstream, so future syncs may conflict on this line — expected and trivial to resolve.

The workflow only triggered on PRs to main, but the fork's default
branch is lime and all fork PRs target it, so the check never ran.
Add lime to the base-branch filter while keeping main.
@adrianschmidt adrianschmidt marked this pull request as ready for review June 17, 2026 12:42
@adrianschmidt adrianschmidt merged commit 927f770 into lime Jun 17, 2026
4 checks passed
@adrianschmidt adrianschmidt deleted the ci/validate-lime-elements-on-fork branch June 17, 2026 12:51
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