Skip to content

Conversation

@lslezak
Copy link
Contributor

@lslezak lslezak commented May 15, 2025

Problem

Solution

  • Specify the permissions
  • In the most cases the minimum contents: read is enough, that allows just reading the Git content and history
  • The Actions which deal with translations needs push permissions and permissions for creating a new pull request

@coveralls
Copy link

coveralls commented May 15, 2025

Coverage Status

coverage: 63.028% (-25.8%) from 88.872%
when pulling c193fcb on action_permissions
into 2e436fc on master.

Copy link
Contributor

@mvidner mvidner left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

@lslezak lslezak merged commit 14f1aaa into master May 15, 2025
23 checks passed
@lslezak lslezak deleted the action_permissions branch May 15, 2025 08:25
@imobachgs imobachgs mentioned this pull request May 26, 2025
imobachgs added a commit that referenced this pull request May 26, 2025
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.

4 participants