Skip to content

fix(ci): use PAT for constraint update commits to trigger CI#5889

Merged
leseb merged 1 commit into
ogx-ai:mainfrom
leseb:leseb/london-v2
May 18, 2026
Merged

fix(ci): use PAT for constraint update commits to trigger CI#5889
leseb merged 1 commit into
ogx-ai:mainfrom
leseb:leseb/london-v2

Conversation

@leseb

@leseb leseb commented May 18, 2026

Copy link
Copy Markdown
Member

What does this PR do?

Fixes CI not running after the constraint-dependencies update workflow commits changes to Dependabot PRs.

The commit-constraint-updates.yml workflow was using github.token (the default GITHUB_TOKEN) for checking out and pushing to same-repo PR branches. GitHub intentionally prevents commits pushed with GITHUB_TOKEN from triggering subsequent workflows to avoid infinite loops. This meant CI never ran on the constraint update commit, leaving PRs like #5859 without status checks.

The fix switches to RELEASE_PAT for both the checkout token: and the git push command on same-repo PRs, matching the fork PR path which already used it correctly.

Test Plan

  • Verify on the next Dependabot PR that constraint updates trigger CI after being committed
  • The fork PR path already uses RELEASE_PAT and works correctly, so this aligns the same-repo path

The commit-constraint-updates workflow was using the default GITHUB_TOKEN
for same-repo PR pushes. Commits pushed with GITHUB_TOKEN do not trigger
subsequent workflow runs (GitHub security feature), so CI never ran after
constraint updates were committed. Switch to RELEASE_PAT for both checkout
and push, matching the fork PR path which already used it.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
Signed-off-by: Sébastien Han <[email protected]>
@leseb
leseb added this pull request to the merge queue May 18, 2026
Merged via the queue into ogx-ai:main with commit 2e9f57f May 18, 2026
15 checks passed
@leseb
leseb deleted the leseb/london-v2 branch May 18, 2026 19:24
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