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
update branch & change base on PRs #921
Conversation
73e7c99
to
7f7c61d
Compare
7f7c61d
to
4b9df79
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to test failure cases, such as when base branch doesn't exist and when expected_head_sha
doesn't match current head).
@tginiotis-at-work |
@bitwiseman Hey, yes. I was a bit busy with other stuff, but I will look into these this week. |
4b9df79
to
4a0d754
Compare
4a0d754
to
de39513
Compare
I have added cases for when the base branch does not exist and when the How does it look now? |
de39513
to
5d7479a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great thanks!
Description
Adding a method to update the PR branch from the base branch & a method to change the base of the PR.
Closes #787.
Before submitting a PR:
We love getting PRs, but we hate asking people for the same basic changes every time.
master
. Create your PR from that branch.mvn clean compile
locally. This may reformat your code, commit those changes.mvn -D enable-ci clean install site
locally. If this command doesn't succeed, your change will not pass CI.When creating a PR: