Skip to content

add explicit perms to CI/CD#20348

Open
seanbudd wants to merge 3 commits into
masterfrom
fixPerm
Open

add explicit perms to CI/CD#20348
seanbudd wants to merge 3 commits into
masterfrom
fixPerm

Conversation

@seanbudd

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings June 16, 2026 05:16
@seanbudd seanbudd requested a review from a team as a code owner June 16, 2026 05:16
@seanbudd seanbudd requested a review from SaschaCowley June 16, 2026 05:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens GitHub Actions security by adding explicit GITHUB_TOKEN permissions to many jobs in the CI/CD workflow, and also fixes a job dependency name typo to match the actual buildNVDA job id.

Changes:

  • Add explicit permissions: blocks to multiple CI jobs (primarily contents: read, plus actions: write where currently specified).
  • Fix uploadSymbols job dependency from buildNvda to buildNVDA.

Comment thread .github/workflows/testAndPublish.yml
Comment thread .github/workflows/testAndPublish.yml
seanbudd and others added 2 commits June 16, 2026 15:33
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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