action-dependabot-auto-merge icon indicating copy to clipboard operation
action-dependabot-auto-merge copied to clipboard

Published 20 hours ago • verified publisher icon ahmadnassri

Define merge strategy

Open mattiaverga opened this issue 1 year ago • 3 comments

Add a strategy flag to decide how to auto-merge the PR:

  • approve-and-merge will mimic actual default, approve the PR and comment with the merge command
  • approve-only will only approve the PR, so that other automatic tools (such as mergify) can merge the PR when CI tests pass
  • merge-only will mimic actual approve: false behavior, only post comment with the merge command without approving the PR

The default approve-and-merge will ensure backward compatibility with existent config files, however unexpected behavior will arise when using strategy: approve-and-merge together with approve: false, therefore when updating the config file the use of approve parameter is deprecated.

Fixes #176

Signed-off-by: Mattia Verga [email protected]

mattiaverga avatar Jan 03 '23 17:01 mattiaverga

@mattiaverga wow, exactly what we need, thank you! @ahmadnassri great feature, will you consider to merge that?

eiskrenkov avatar Jan 20 '23 17:01 eiskrenkov

@ahmadnassri Sorry for the ping, but could you please review this?

yeikel avatar Feb 22 '23 05:02 yeikel

apologies for the delay, been dealing with personal matters and have not looked at code in over a month...

confident I'll land this PR this week as part of a major rewrite

ahmadnassri avatar Feb 22 '23 12:02 ahmadnassri