Skip to content

feat(ci): run verify plugin export workflow on pull requests#145

Merged
christoph-jerolimov merged 3 commits into
mainfrom
ci/verify-on-pr
Jul 22, 2026
Merged

feat(ci): run verify plugin export workflow on pull requests#145
christoph-jerolimov merged 3 commits into
mainfrom
ci/verify-on-pr

Conversation

@christoph-jerolimov

@christoph-jerolimov christoph-jerolimov commented Jul 20, 2026

Copy link
Copy Markdown
Member

Summary

  • Add pull_request trigger to the verify plugin export workflow so it runs on every PR

Test plan

  • Verify the workflow triggers on this PR itself

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@christoph-jerolimov christoph-jerolimov changed the title ci: run verify plugin export workflow on pull requests feat(ci): run verify plugin export workflow on pull requests Jul 20, 2026
christoph-jerolimov and others added 2 commits July 20, 2026 11:52
Use empty string fallback for rhdh-cli checkouts so actions/checkout
uses its default behavior (PR merge commit for PRs, default branch
for schedule). Use 'main' fallback for the overlays repo checkout.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use github.base_ref for PRs so the overlays repo checks out the
matching target branch (e.g. release-1.10) instead of always main.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@sonarqubecloud

Copy link
Copy Markdown

@gashcrumb gashcrumb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, if we have this per-pr should the concurrency section still be commented out?

@christoph-jerolimov

Copy link
Copy Markdown
Member Author

If it runs to often we can change that.

@christoph-jerolimov
christoph-jerolimov merged commit 502b05a into main Jul 22, 2026
17 checks passed
@christoph-jerolimov
christoph-jerolimov deleted the ci/verify-on-pr branch July 22, 2026 21:12
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