Skip to content

Debug#1048

Open
tstellar wants to merge 22 commits into
mainfrom
subscriber-fix
Open

Debug#1048
tstellar wants to merge 22 commits into
mainfrom
subscriber-fix

Conversation

@tstellar
Copy link
Copy Markdown
Owner

No description provided.

tstellar added 2 commits May 29, 2026 14:14
This consolidates duplicate logic from the pr-subscriber and
issue-subscriber workflows into a new workflow called
subscriber-write.  This workflow uses the workflow_run trigger
and is triggered by another new workflow called subscriber.

The subscriber workflow collects the label name and issue number
from the labeled issue/pull requests and sends it to the
subscriber-write workflow via artifacts.

Besides removing some duplicate code this change also allows us to
switch from using the pull_request_target event to the pull_request
event for handling pull request subscribers.  pull_request executes
in an untrusted context and helps protect us from some security risks.
@tstellar tstellar force-pushed the main branch 2 times, most recently from fb27c21 to 932bb3c Compare May 31, 2026 18:47
@tstellar tstellar force-pushed the main branch 2 times, most recently from 134eb98 to 60fcfd0 Compare May 31, 2026 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant