Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ability to backport a PR to any repo, not necessarily a fork #384

Closed
mvayngrib opened this issue Sep 7, 2023 · 3 comments · Fixed by #405
Closed

ability to backport a PR to any repo, not necessarily a fork #384

mvayngrib opened this issue Sep 7, 2023 · 3 comments · Fixed by #405
Labels
enhancement New feature or request

Comments

@mvayngrib
Copy link

Is your feature request related to a problem? Please describe.

forks get detached for one reason or another, but backporting between repos can still be useful

Describe the solution you'd like

a github repo option, e.g. target_repo(s), to specify to which repo to send the backport PR

Describe alternatives you've considered
not sure what the alternatives would be 😅

@mvayngrib mvayngrib added the enhancement New feature or request label Sep 7, 2023
@korthout
Copy link
Owner

korthout commented Sep 8, 2023

That is a great idea! Thanks for raising it @mvayngrib 🙌

The target_repo input would take something in the form of owner/repo defaulting to the current repo on which the action is run.

This feature would require changing the following:

  • add an extra remote for the target repo, i.e. target remote
  • git fetch of target branch from target remote
  • git push of backport branch to target remote
  • create pull request on target repo
  • label pull request on target repo

@mvayngrib
Copy link
Author

mvayngrib commented Sep 12, 2023

i was thinking about it and it might make more sense to support a field targets of the format owner1/repo1/branch1,owner2/repo2/branch2 to avoid getting confused with combinations of target_repo and target_branches

alternatively, we add support via labels first (maybe via named capturing groups?), and see if anyone else cares about this, then add targets later

@korthout
Copy link
Owner

Resolved by #405 and released in v3.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants