Open mattiaverga opened 1 year ago
@mattiaverga wow, exactly what we need, thank you! @ahmadnassri great feature, will you consider to merge that?
@ahmadnassri Sorry for the ping, but could you please review this?
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
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 commandapprove-only
will only approve the PR, so that other automatic tools (such as mergify) can merge the PR when CI tests passmerge-only
will mimic actualapprove: false
behavior, only post comment with the merge command without approving the PRThe default
approve-and-merge
will ensure backward compatibility with existent config files, however unexpected behavior will arise when usingstrategy: approve-and-merge
together withapprove: false
, therefore when updating the config file the use ofapprove
parameter is deprecated.Fixes #176
Signed-off-by: Mattia Verga mattia.verga@tiscali.it