GraphqlError: Base branch was modified. Review and try the merge again. about merge-me-action

[z0rc]

While we're at it I think check shouldn't succeed on internal failure.

[mad-it]

Sorry for taking a bit longer to respond. This will require a bit more of an internal discussions.

There are quite a few cases where we want the errors to be swallowed and not retried. Here are a few examples for our own usages:
##[error]GraphqlError: 5 of 5 required status checks have not succeeded: . - This is an example of when a build is still in progress.
##[error]GraphqlError: 4 of 4 required status checks are expected. - An alternative scenario where a build is still in progress.

So we need to discuss whether we will retry for the specific case you mentioned or come up with a more general solution.

While we're at it I think check shouldn't succeed on internal failure.

I do agree with this, however GitHub actions still does not support restarting individual jobs, so failing the automated check would require you to restart the entire build. So that will be a somewhat nasty side effect. I will take this up as well as the intention of this action is to merge the PR. If that fails it should indeed indicate as much.

[mad-it]

While we're at it I think check shouldn't succeed on internal failure.

This part has been resolved by #469

The rest is pending some discussions :)

[ricardoatsouza]

Hello @z0rc. Retry on failure for this error was added in this PR: #480. Hope this solves the problem you were having. :)