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

Add pull-requests read permission to the release-notes-check workflow #289

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

llucax
Copy link
Contributor

@llucax llucax commented Jul 10, 2024

This is necessary when the repository Actions configuration is set up without full access by default. In this case, the only access provided is contents: read, but this action needs to read the pull request too.

@llucax llucax requested a review from a team as a code owner July 10, 2024 10:39
@llucax llucax requested a review from shsms July 10, 2024 10:39
@leandro-lucarella-frequenz leandro-lucarella-frequenz added this to the v0.10.0 milestone Jul 10, 2024
@github-actions github-actions bot added part:ci Affects the GitHub workflow and other parts for running CI part:template Affects the cookiecutter template files labels Jul 10, 2024
This is necessary when the repository Actions configuration is set up
without full access by default. In this case, the only access provided
is `contents: read`, but this action needs to read the pull request
too.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax llucax force-pushed the fix-rel-notes-check branch from 6a5c2d8 to 1778ff8 Compare July 10, 2024 10:41
@llucax llucax added the type:bug Something isn't working label Jul 10, 2024
@llucax llucax enabled auto-merge July 10, 2024 10:42
Copy link

@flora-hofmann-frequenz flora-hofmann-frequenz left a comment

Choose a reason for hiding this comment

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

LGTM :-)

@llucax llucax added this pull request to the merge queue Jul 10, 2024
Merged via the queue into frequenz-floss:v0.x.x with commit bfd45a8 Jul 10, 2024
15 checks passed
@llucax llucax deleted the fix-rel-notes-check branch July 10, 2024 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
part:ci Affects the GitHub workflow and other parts for running CI part:template Affects the cookiecutter template files type:bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants