Skip to content

Expand /cherry-pick workflow eligibility to project committers#7343

Draft
smlambert with Copilot wants to merge 7 commits into
masterfrom
copilot/improve-cherry-pick-workflow
Draft

Expand /cherry-pick workflow eligibility to project committers#7343
smlambert with Copilot wants to merge 7 commits into
masterfrom
copilot/improve-cherry-pick-workflow

Conversation

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

The /cherry-pick workflow was too restrictive: it allowed only PR author/reviewer/merger roles, which blocked some repository committers from triggering cherry-picks. This updates authorization to align with maintainer workflow needs while keeping PR-author access intact.

  • Permission model update (.github/workflows/cherry-pick.yml)

    • Replaced reviewer/merger-based checks with:
      • PR author allowlist
      • repository collaborator permission check (write/maintain/admin)
    • Added clearer handling for collaborator permission lookup failures and explicit fallback behavior for non-eligible users.
  • User-facing behavior/docs alignment (docs/pages/userGuide.md)

    • Updated /cherry-pick command documentation to reflect the new eligibility model: PR author or project committer.
  • Authorization logic (example)

    case "$PERMISSION" in
      admin|maintain|write)
        echo "✅ Commenter is a project committer with $PERMISSION permission"
        exit 0
        ;;
      *)
        ;;
    esac

Copilot AI linked an issue Jul 10, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Update cherry-pick workflow permissions for project committers Expand /cherry-pick workflow eligibility to project committers Jul 10, 2026
Copilot AI requested a review from smlambert July 10, 2026 17:45
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.

Improvements to /cherry-pick workflow

2 participants