WeTransfer/WeTransfer-iOS-CI

[CI] Add an option to automatically merge PRs on green

AvdLee opened this issue · 8 comments

Automatically merging PRs would speed up quite some cases

@BasThomas would love to get your ideas for this! Any experience with solutions?

Haven't implemented something like this in the past. But let me take a look at it; especially the built-in GitHub automerge.

This is fixed by enabling auto merge on our repositories.

Reopening since auto-merge is also merging for red PRs, which is not good. It should really only merge when CI is green!

Oh noes! 🙈

It seems to be fixed by setting restrictions towards merging, e.g. Danger and Bitrise need to succeed.
This is now enabled:
image

The downside is that manual merging isn't possible either when CI failed. Though, we really don't want to auto-merge when checks fail. Let's see whether these restrictions really become a problem.

Closing for now, lets reopen if we need to revisit our implementation.