Skip to content

doc: update contributing

Pascal Engélibert requested to merge tuxmain-contributing into master

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

Merge request reports