Skip to content

fix(ci): remove pull_request trigger from Go Dependency Submission workflow#444

Merged
diranged merged 1 commit intomainfrom
fix/godeps-workflow-permissions
Mar 11, 2026
Merged

fix(ci): remove pull_request trigger from Go Dependency Submission workflow#444
diranged merged 1 commit intomainfrom
fix/godeps-workflow-permissions

Conversation

@diranged
Copy link
Copy Markdown
Owner

Summary

  • The godeps.yaml workflow triggers on pull_request events, but the actions/go-dependency-submission action requires contents: write permission to submit dependency snapshots
  • On pull_request events, the GITHUB_TOKEN is read-only, causing "Resource not accessible by integration" errors (e.g. PR #443)
  • Dependency snapshots only need to be submitted when code lands on main, so the pull_request trigger is removed

Test plan

  • Verify CI passes on this PR (the go-action-detection job should no longer run)
  • Verify the workflow still runs on push to main after merge

🤖 Generated with Claude Code

…rkflow

The go-dependency-submission action requires `contents: write` permission
to submit dependency snapshots, but `pull_request` events provide a
read-only GITHUB_TOKEN. This causes "Resource not accessible by
integration" errors on PRs. Since dependency snapshots only need to be
submitted when code lands on main, the `pull_request` trigger is
unnecessary.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the github_actions Pull requests that update GitHub Actions code label Mar 11, 2026
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 22932512733

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 36.007%

Totals Coverage Status
Change from base Build 22777299803: 0.0%
Covered Lines: 1037
Relevant Lines: 2880

💛 - Coveralls

@diranged diranged marked this pull request as ready for review March 11, 2026 02:15
@diranged diranged merged commit 673f231 into main Mar 11, 2026
21 of 33 checks passed
@diranged diranged deleted the fix/godeps-workflow-permissions branch March 11, 2026 02:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants