doc: update contributing
Update contributing guide.
Je pense qu'il faudrait surtout clarifier ceci : en tant que relecteur, comment déterminer si l'auteur d'une MR veut qu'elle soit relue, mergée. Voici mon algo personnel en cas de MR:
- si Draft et je ne suis pas reviewer, je n'y touche pas.
- si Draft et je suis reviewer, je relis mais pas forcément dans le détail puisque ça peut encore changer. Je ne retire pas le Draft.
- si non Draft et je suis reviewer, je relis attentivement, et éventuellement accepte, auquel cas si de plus l'auteur n'a pas le droit de merger, je merge.
- s'il reste des threads non résolus, je n'accepte pas.
- si non Draft et aucun reviewer, l'auteur doit être Chuck Norris, donc je n'y touche pas.
Tout commentaire explicite de l'auteur ayant la priorité sur l'algo.
Est-ce que ça convient à tout le monde ? Est-ce que ça vaut le coup d'insister là-dessus ?
Edited by Pascal Engélibert