Skip to content

Harden sglang sync workflow against PR token restrictions#2

Draft
jdai0 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job
Draft

Harden sglang sync workflow against PR token restrictions#2
jdai0 with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-failing-github-actions-job

Conversation

Copilot AI commented Jun 11, 2026

Copy link
Copy Markdown

The Check for sglang-kt updates job was failing when repository policy disallowed PR creation via GITHUB_TOKEN. This change keeps the sync check usable in restricted repos while preserving automated PR behavior when elevated credentials are available.

  • Token strategy

    • Introduce SYNC_PR_TOKEN with fallback: SGLANG_SYNC_TOKEN (PAT) → GITHUB_TOKEN.
    • Use the same token for checkout and create-pull-request to avoid mixed-auth behavior.
  • Failure handling

    • Make PR creation non-fatal (continue-on-error: true) for update-check runs.
    • Add explicit warning output when PR creation is blocked, with actionable guidance to configure SGLANG_SYNC_TOKEN.
  • Workflow scope

    • Changes are isolated to .github/workflows/sync-sglang-submodule.yml; no application/runtime code paths were modified.
env:
  SYNC_PR_TOKEN: ${{ secrets.SGLANG_SYNC_TOKEN || secrets.GITHUB_TOKEN }}

- uses: peter-evans/create-pull-request@v6
  id: create_pr
  continue-on-error: true
  with:
    token: ${{ env.SYNC_PR_TOKEN }}

Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'Check for sglang-kt updates' Harden sglang sync workflow against PR token restrictions Jun 11, 2026
Copilot finished work on behalf of jdai0 June 11, 2026 12:08
Copilot AI requested a review from jdai0 June 11, 2026 12:08
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