Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

only give read permissions in GitHub Actions workflows #4182

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

boegel
Copy link
Member

@boegel boegel commented Jan 18, 2023

By default, CI workflows get full permissions (incl. write) to the repo, there's absolutely no need for this in our case.

cfr. https://docs.github.com/en/actions/using-jobs/assigning-permissions-to-jobs

@ocaisa
Copy link
Member

ocaisa commented Jan 18, 2023

Yeah, this stuff is important. One thing to note is that you can set default permissions for the organisation (but it is also good to be explicit)

@ocaisa
Copy link
Member

ocaisa commented Jan 18, 2023

@ocaisa ocaisa merged commit bfa26c9 into easybuilders:develop Jan 18, 2023
@boegel boegel deleted the read_only_ci_workflows branch January 18, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants