Skip to content

Confirm build.yaml permissions are correct (no pull-requests: write needed)#25

Closed
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-24
Closed

Confirm build.yaml permissions are correct (no pull-requests: write needed)#25
Copilot wants to merge 2 commits intomainfrom
copilot/sub-pr-24

Conversation

Copy link
Contributor

Copilot AI commented Dec 25, 2025

Addresses review feedback on PR #24 questioning whether build.yaml requires pull-requests: write permission for uploading artifacts.

Analysis

The build.yaml workflow does not need pull-requests: write because:

  • actions/upload-artifact@v6 stores artifacts in workflow run storage, not on pull requests
  • No actions create PR comments, update PR status, or modify PR metadata
  • Only read operations on repository contents are performed

Recommendation

Current contents: read permission is correct and follows least privilege principle. No changes required.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Malcolmnixon and others added 2 commits December 24, 2025 21:21
…n permissions

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix permissions in code scanning alert process Confirm build.yaml permissions are correct (no pull-requests: write needed) Dec 25, 2025
Copilot AI requested a review from Malcolmnixon December 25, 2025 02:25
Base automatically changed from alert-autofix-3 to main December 25, 2025 02:26
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