Skip to content

Fix board pre-validation workflow permission for PR comment publishing#797

Merged
ADubeyAMD merged 2 commits into
2024.2from
copilot/fix-validate-board-files-again
Jun 4, 2026
Merged

Fix board pre-validation workflow permission for PR comment publishing#797
ADubeyAMD merged 2 commits into
2024.2from
copilot/fix-validate-board-files-again

Conversation

Copilot AI commented Jun 4, 2026

Copy link
Copy Markdown

The Validate Board Files job was failing despite successful validation because the PR comment step hit 403 Resource not accessible by integration. The workflow had pull-requests: write but lacked permission required for Issue Comments API writes.

  • Root cause

    • actions/github-script posts to POST /repos/{owner}/{repo}/issues/{number}/comments
    • Workflow token did not include issues: write
  • Change

    • Updated workflow permissions in .github/workflows/board_pr_validation.yml to grant issue comment write access.
  • Result

    • Validation can continue posting/updating board validation reports on PRs without failing due to token scope mismatch.
permissions:
  contents: read
  issues: write
  pull-requests: write

@xilinxgitops

Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

Copilot AI changed the title [WIP] Fix failing GitHub Actions job Validate Board Files Fix board pre-validation workflow permission for PR comment publishing Jun 4, 2026
Copilot AI requested a review from ADubeyAMD June 4, 2026 08:19
@github-actions

github-actions Bot commented Jun 4, 2026

Copy link
Copy Markdown

🔍 Board Pre-Validation Report

No board files (under boards/) were changed in this pull request.

@ADubeyAMD ADubeyAMD marked this pull request as ready for review June 4, 2026 08:25
@ADubeyAMD ADubeyAMD merged commit 48a3839 into 2024.2 Jun 4, 2026
1 check passed
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