More warnings for MR reviewer

Now we warn reviewer about not passed CI but I think there are two more cases when we should warn reviewer.

  • Too many commits in MR. More than 10 is a bit suspicious. It would help to not overlook 190 commits here https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5049
  • Presence of commits that have more than one parent commit. In other words I mean warning about non-linear history.

Humans tend to overlook stuff so let's make it impossible.

/cc @JobV @dzaporozhets @DouweM @rspeicher

Edited Sep 02, 2020 by 🤖 GitLab Bot 🤖
Assignee Loading
Time tracking Loading