Skip to content

Simplify auto-backport.yml: remove workflow_dispatch, match working fork - #47

Merged
bunnam988 merged 1 commit into
rdkcentral:developfrom
bunnam988:fix/simplify-auto-backport
Aug 7, 2026
Merged

Simplify auto-backport.yml: remove workflow_dispatch, match working fork#47
bunnam988 merged 1 commit into
rdkcentral:developfrom
bunnam988:fix/simplify-auto-backport

Conversation

@bunnam988

Copy link
Copy Markdown
Contributor

Removes unnecessary workflow_dispatch trigger. The simple pull_request:labeled/closed pattern already works for post-merge labels (confirmed on bunnam988 fork). Matches the exact tested working pattern from the fork.

Reason for change: Simplify to tested working pattern
Test Procedure: Merge PR, add cherry-pick label → workflow fires automatically
Risks: Low
Priority: P1

…pattern

Reason for change: pull_request:labeled fires for merged PRs; simpler is better
Test Procedure: merge PR to develop, add cherry-pick label — workflow triggers automatically
Risks: Low
Priority: P1
@bunnam988
bunnam988 requested review from a team as code owners August 7, 2026 10:10
Copilot AI review requested due to automatic review settings August 7, 2026 10:10
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

📋 PR Format Reminder

  • Title: Simplify auto-backport.yml: remove workflow_dispatch, match working fork — expected TICKET-123 : description
    (Multiple tickets OK: RDKCOM-5492 RDKBDEV-3336 : ... | Include US ticket + subtask for user-stories)

Expected:

TICKET-123 : brief description

Reason for change: why
Test Procedure: how to verify
Risks: Low / Medium / High
Priority: P0 / P1 / P2

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR simplifies the auto-backport GitHub Actions workflow by removing the workflow_dispatch trigger and related input-handling logic, aiming to match the proven “post-merge label triggers workflow” pattern.

Changes:

  • Removed workflow_dispatch trigger and its pr_number / target_branch inputs.
  • Simplified the job if: condition to only run for merged PRs with a cherry-pick to label.
  • Removed conditional label/PR override logic tied to manual dispatch inputs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/auto-backport.yml
@bunnam988
bunnam988 merged commit 25fd2a1 into rdkcentral:develop Aug 7, 2026
7 of 8 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 7, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants