Skip to content

make --merge-pr take into account --pr-target-branch#3236

Merged
akesandgren merged 2 commits intoeasybuilders:developfrom
boegel:merge_pr_target_branch
Mar 8, 2020
Merged

make --merge-pr take into account --pr-target-branch#3236
akesandgren merged 2 commits intoeasybuilders:developfrom
boegel:merge_pr_target_branch

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented Mar 7, 2020

While merging easyconfigs PRs to the 2020a branch, I'm quite annoyed that --merge-pr fails because the target branch isn't develop.

So, let's make --merge-pr honor --pr-target-branch, so we can merge PRs that target 2020a using eb --pr-target-branch 2020a --merge-pr...

Of course the default is still develop (which is the default value for --pr-target-branch).

Copy link
Copy Markdown
Contributor

@akesandgren akesandgren left a comment

Choose a reason for hiding this comment

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

LGTM

@akesandgren
Copy link
Copy Markdown
Contributor

Going in, thanks @boegel!

@akesandgren akesandgren merged commit b955832 into easybuilders:develop Mar 8, 2020
@boegel boegel deleted the merge_pr_target_branch branch March 8, 2020 08:40
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.

2 participants