Skip to content

ci: Use least privilege principle for permissions #609

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
May 13, 2025
Merged

ci: Use least privilege principle for permissions #609

merged 1 commit into from
May 13, 2025

Conversation

flovogt
Copy link
Member

@flovogt flovogt commented May 12, 2025

Declares only the required permissions per workflow.

@flovogt
ci: Use least privilege principle for permissions
c79df35 
Declares only the required permissions per workflow.
@flovogt flovogt requested a review from a team May 12, 2025 13:02
@coveralls
Copy link

Coverage Status

coverage: 90.231%. remained the same
when pulling c79df35 on ci-permissions
into 4c4943c on main.

@flovogt flovogt merged commit f710f75 into main May 13, 2025
18 checks passed
@flovogt flovogt deleted the ci-permissions branch May 13, 2025 07:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@d3xter666 d3xter666 d3xter666 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@flovogt @coveralls @d3xter666