Support for rebasing before merging PR about bulldozer HOT 7 OPEN

I think that is a reasonable feature request. You are correct in that this isn't currently supported behaviour. While we don't have time right now to make the change, we accept PRs and will review requests in a timely manner.

I think if you wanted to take a shot at this, you'd want to to add the merge_method param to the update configuration, and then using that merge method https://github.com/palantir/bulldozer/blob/develop/bulldozer/update.go#L102

from bulldozer.

Hi @arunpkqt ,

I guess we need to highlight this better in the docs, but the answer is yes! In the spec we allow users to specify the merge method as either rebase, squash or merge.

You can also specify method by branch if you want to treat develop/master/release different from one another.

from bulldozer.

It looks like this setting configures the merge method when merging the branch to the target branch, but what I'd like to configure is the merge method when updating the branch from the target branch. This avoids the confusing git history when using merges to update the branch from the target branch.

I think this would mean that update: can be set to method: rebase

Perhaps this is already possible and I don't understand the configuration?

Thanks!

from bulldozer.

On review, I don't think this is possible with the current Bulldozer architecture. The branch update feature relies on the Perform a merge API to do updates without making any local clones, and I believe that API can only create regular merge commits.

I don't want to add the complexity of managing local clones to Bulldozer, so I think this is blocked either on a new feature in GitHub or a clever way to perform a rebase using only the API, maybe via the Git Data APIs.

from bulldozer.

That makes sense. I doubt that GitHub is going to add a rebase branch API any time soon, but I'll email them a feature request anyway. Thanks for having a look.

from bulldozer.

There is a similar request on Kodiak (chdsbd/kodiak#150) -- it looks like there may be a way to do this with just the GitHub API (chdsbd/kodiak#150 (comment)).

from bulldozer.

This feature would be great to have!
Any updates?

from bulldozer.