elastic / jarvis

Logstash's ChatOps bot, J.A.R.V.I.S.
41 stars 28 forks source link

Require flag to merge PRs with multiple commits #97

Open jsvd opened 5 years ago

jsvd commented 5 years ago

Following the comment https://github.com/elastic/jarvis/issues/8#issuecomment-113250576, Jarvis should refuse to merge a PR if it has more than one commit. One should also have the ability to overrule this behaviour with a flag like --allow-multi.

kares commented 4 years ago

was thinking about the alternative to do a "squash and merge" GH call and after that cherry-pick to additional branches specified ... might be a new habit to learn :wind_chime: