Skip to content

Use pull_request instead of pull_request_target, with minimal permission#78

Merged
lbroudoux merged 2 commits intomicrocks:mainfrom
yada:main
Mar 22, 2026
Merged

Use pull_request instead of pull_request_target, with minimal permission#78
lbroudoux merged 2 commits intomicrocks:mainfrom
yada:main

Conversation

@yada
Copy link
Member

@yada yada commented Mar 20, 2026

fix #77

…ions

Signed-off-by: Yacine Kheddache <yacine@microcks.io>
@yada yada added enhancement New feature or request github_actions Pull requests that update GitHub Actions code labels Mar 20, 2026
Signed-off-by: Yacine Kheddache <yacine@microcks.io>
Copy link
Member

@lbroudoux lbroudoux left a comment

Choose a reason for hiding this comment

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

LGTM

@lbroudoux lbroudoux merged commit 3848eb0 into microcks:main Mar 22, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

GitHub Actions Security: Avoiding pull_request_target Pitfalls in welcome-new-users.yml

3 participants