Skip to content

bender needs to fire on synchronize also - #78

Merged
cgoea merged 3 commits into
developfrom
users/cgoea/bender
Aug 13, 2026
Merged

bender needs to fire on synchronize also#78
cgoea merged 3 commits into
developfrom
users/cgoea/bender

Conversation

@cgoea

@cgoea cgoea commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Motivation

Bender needs to fire on synchronize event also, otherwise it misses new pushes to the PR branch

Submission Checklist

@cgoea
cgoea requested a review from a team August 13, 2026 16:57
cgoea added 2 commits August 13, 2026 19:58
…ew pushes

Bender only ran on opened/edited/reopened, so pushing new commits to an
existing PR left no check for the new head sha. Since Bender is a required
check, GitHub then blocks merging on a missing (not failing) check.
Removed unnecessary comments from pull request trigger.
@cgoea
cgoea changed the base branch from main to develop August 13, 2026 17:01
@cgoea
cgoea merged commit c9470b0 into develop Aug 13, 2026
2 of 3 checks passed
@cgoea
cgoea deleted the users/cgoea/bender branch August 13, 2026 18:40
quartz-sync-github-app Bot pushed a commit that referenced this pull request Aug 13, 2026
c9470b0, bender needs to fire on synchronize also (#78), Ciprian Goea (ciprian.goea@amd.com), Thu Aug 13 21:40:20 2026 +0300
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