Skip to content

Fix CI not triggering on automated API update PRs#585

Merged
dblock merged 1 commit intomasterfrom
copilot/sub-pr-578
Mar 3, 2026
Merged

Fix CI not triggering on automated API update PRs#585
dblock merged 1 commit intomasterfrom
copilot/sub-pr-578

Conversation

Copy link
Contributor

Copilot AI commented Mar 3, 2026

PRs created by the update_api workflow never triggered CI because the create-pull-request step used GITHUB_TOKEN — GitHub intentionally suppresses workflow triggers on events from that token to prevent infinite loops.

Change

Use the GitHub App token (already obtained earlier in the workflow) for the create-pull-request step, falling back to GITHUB_TOKEN when unavailable:

# Before
token: ${{ secrets.GITHUB_TOKEN }}

# After
token: ${{ steps.github_app_token.outputs.token || secrets.GITHUB_TOKEN }}

This is consistent with the pattern already used in the subsequent "Commit and Push" step.


🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

@dblock
Copy link
Collaborator

dblock commented Mar 3, 2026

@copilot rebase against master branch

Copilot AI changed the title [WIP] Update API from slack-api-ref at revision 3275786 Fix CI not triggering on automated API update PRs Mar 3, 2026
Copy link
Contributor Author

Copilot AI commented Mar 3, 2026

@copilot rebase against master branch

I'm unable to perform a rebase — my environment doesn't support force pushes, which are required to update the branch after rewriting commit history with git rebase. You can rebase this branch onto master manually with:

git fetch origin
git checkout copilot/sub-pr-578
git rebase origin/master
git push --force-with-lease origin copilot/sub-pr-578

Alternatively, the PR base branch can be changed from automated-api-update to master via the GitHub UI, since my only change (update_api.yml) applies cleanly to master.

@dblock dblock force-pushed the copilot/sub-pr-578 branch from 726f636 to 75c6af9 Compare March 3, 2026 20:54
@github-actions
Copy link

github-actions bot commented Mar 3, 2026

Danger Report

No issues found.

View run

@dblock dblock changed the base branch from automated-api-update to master March 3, 2026 20:55
@dblock dblock force-pushed the copilot/sub-pr-578 branch from 75c6af9 to 81bfd89 Compare March 3, 2026 20:56
… token

Co-authored-by: dblock <542335+dblock@users.noreply.github.com>
@dblock dblock force-pushed the copilot/sub-pr-578 branch from 81bfd89 to 040a076 Compare March 3, 2026 20:57
@dblock dblock marked this pull request as ready for review March 3, 2026 20:57
@dblock dblock merged commit 891aecf into master Mar 3, 2026
19 checks passed
@dblock dblock deleted the copilot/sub-pr-578 branch March 3, 2026 20:58
@coveralls
Copy link

coveralls commented Mar 3, 2026

Pull Request Test Coverage Report for Build 22642386833

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 91.002%

Totals Coverage Status
Change from base Build 22503147252: 0.0%
Covered Lines: 5057
Relevant Lines: 5557

💛 - Coveralls

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.

3 participants