Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

chore: explicit checks permissions #17

Merged
merged 2 commits into from
Jan 30, 2024
Merged

chore: explicit checks permissions #17

merged 2 commits into from
Jan 30, 2024

Conversation

RyanZim
Copy link
Collaborator

@RyanZim RyanZim commented Jan 26, 2024

No description provided.

@RyanZim RyanZim self-assigned this Jan 26, 2024
Copy link
Contributor

@mvayngrib mvayngrib left a comment

Choose a reason for hiding this comment

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

maybe move it to the top so new tasks default to read?

@RyanZim
Copy link
Collaborator Author

RyanZim commented Jan 26, 2024

maybe move it to the top so new tasks default to read?

I thought the point was to only grant access where needed. Granted, unlikely you'd add a new task here that doesn't require read, but still.

@RyanZim RyanZim merged commit 94441f8 into master Jan 30, 2024
2 checks passed
@RyanZim RyanZim deleted the ryan/ci-perms branch January 30, 2024 16:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants