Kevin Intriago
04/17/2026, 2:20 PMPylon
04/17/2026, 2:20 PMgt feedback --with-debug-context <any message>Pylon
04/17/2026, 2:22 PMKevin Intriago
04/17/2026, 2:23 PMPylon
04/17/2026, 2:40 PMKevin Intriago
04/17/2026, 2:58 PMon:
push:
branches:
- main
pull_request:
types:
- closed
- opened
- synchronize
branches-ignore:
- "**/graphite-base/**"Kevin Intriago
04/17/2026, 2:59 PMKevin Intriago
04/17/2026, 3:38 PMPylon
04/17/2026, 3:47 PMopened, synchronize, closed. when graphite retargets a branch, there's a possibility that github fires a pull_request.edited event rather than synchronize. if that's what's happening, adding edited would make CI trigger on the retarget. edited also fires for PR title/description edits, so CI will re-run on those too. if that's a concern, adding a condition so your jobs only run when the base actually changes should fix this.Kevin Intriago
04/17/2026, 3:50 PMedited. This action is triggered when metadata about the PR is changed, not the PR itself (that would be synchronize). The edited event triggers a CI run when the PR below a given PR is merged—as the change of the base branch from the downstack PR to trunk is considered an “edit”.
I think our main concern here is if thats the actual expected behaviour, I mean if the recommendation is to avoid graphite base branches, then why we have this behaviour? perhaps that is missing in the docs besides the advice to filter out the graphite base branches? 🤔 We are a bit confusedKevin Intriago
04/17/2026, 3:51 PMPylon
04/17/2026, 4:23 PMedited to the types as stated in our graphite docs. it seems like the issue is that there seems to be some flakiness with Github Actions where force pushing doesn't deliver the synchronize event -- which it should. we do have an internal ticket in place for this issue that is currently on our engineers radar.Kevin Intriago
04/17/2026, 4:35 PM