Skip to content

Support pull_request_target event - #3

Merged
llucax merged 2 commits into
frequenz-floss:v1.x.xfrom
llucax:pr_target
Feb 17, 2026
Merged

Support pull_request_target event#3
llucax merged 2 commits into
frequenz-floss:v1.x.xfrom
llucax:pr_target

Conversation

@llucax

@llucax llucax commented Feb 17, 2026

Copy link
Copy Markdown

Handle pull_request_target alongside pull_request when detecting Dependabot PRs, fetching metadata, and reading titles. This keeps manual dispatch logic unchanged while enabling target-context runs.

Also avoid direct interpolation of GitHub expressions inside shell scripts to reduce the risk of unescaped external data.

Avoid direct interpolation of GitHub expressions inside shell scripts by
mapping inputs and context values through env variables. This reduces
the risk of unescaped external data and keeps shell logic consistent
across events and manual dispatches.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
Handle `pull_request_target` alongside `pull_request` when detecting
Dependabot PRs, fetching metadata, and reading titles. This keeps
manual dispatch logic unchanged while enabling target-context runs.

Signed-off-by: Leandro Lucarella <luca-frequenz@llucax.com>
@llucax
llucax requested a review from Marenz February 17, 2026 11:17
@llucax
llucax enabled auto-merge February 17, 2026 11:22
@llucax
llucax disabled auto-merge February 17, 2026 11:25
@llucax

llucax commented Feb 17, 2026

Copy link
Copy Markdown
Author

@llucax
llucax enabled auto-merge February 17, 2026 13:28
@llucax
llucax merged commit 98f4b7f into frequenz-floss:v1.x.x Feb 17, 2026
1 check passed
@llucax
llucax deleted the pr_target branch February 18, 2026 11:26
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.

2 participants