Yicong-Huang opened a new issue, #4605:
URL: https://github.com/apache/texera/issues/4605

   ### Background
   
   We currently have two separate workflows that react to `pull_request_target` 
on PR open:
   
   - `Pull Request Labeler / labeler` — applies labels via `actions/labeler`.
   - `Auto-assign / assign-pr-author` — assigns the PR author to the PR.
   
   Having two workflows produce two separate check entries on every PR adds 
noise without functional value, since both run the same trigger and have the 
same audience.
   
   There is also a third job, `Auto-assign / credit-issue-on-pr-merge`, which 
is unrelated to PR open — it runs on PR close and credits authors on the linked 
issue.
   
   ### Proposal
   
   - Move the `assign-pr-author` logic into the existing `Pull Request Labeler 
/ labeler` job as an additional step (gated on `action == 'opened'`).
   - Reduce `Auto-assign` to just `credit-issue-on-pr-merge`, triggered only by 
`pull_request_target: closed` and gated on `pull_request.merged == true`.
   
   This keeps all PR-open automation under one check, and keeps the merge-time 
credit logic isolated and clearly scoped to merged PRs.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to