Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Minimize permissions to CI workflows #133

Merged
merged 1 commit into from
Mar 20, 2023
Merged

Conversation

sywhang
Copy link
Contributor

@sywhang sywhang commented Mar 20, 2023

Set only read permission on CI workflows since they don't need write access.

Fixes #132.

Set only read permission on CI workflows since they don't need write access.

Fixes uber-go#132.
@sywhang sywhang requested review from r-hang and tchung1118 March 20, 2023 13:34
@codecov
Copy link

codecov bot commented Mar 20, 2023

Codecov Report

Merging #133 (49806f2) into master (1505d28) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master      #133   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines          242       242           
=========================================
  Hits           242       242           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@r-hang r-hang merged commit 544d6aa into uber-go:master Mar 20, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Set minimum permissions to workflows
3 participants