web-dev-qa-db-fra.com

Est-il correct de demander aux contributeurs de rebaisser leurs demandes de traction sur GitHub

Je maintiens un repo relativement populaire GitHub.

Lorsqu'une demande de traction est bonne de fusionner, je demande généralement à l'auteur de la repasser à un seul commit avant de la fusionner (surtout quand il y a eu plusieurs petites modifications).

Est-ce une bonne pratique GIT? Est-ce que cette étiquette Github acceptable/standard?

Donc quelques avantages:

  • Je reçois une belle histoire de commettre propre dans les journaux de validation
  • Je n'ai pas besoin de changer le commettre moi-même
  • Il délègue une partie du travail

Quelques inconvénients possibles:

  • Je ne sais pas si c'est une bonne étiquette
  • Je ne sais pas si c'est une bonne pratique git
  • J'ai généralement déjà demandé quelques autres changements - c'est un de plus et je ne veux pas décourager les contributeurs.
25
Pablojim

En ce qui concerne le Git, c'est une guerre sainte si vous devriez simplement fusionner des succursales ou la Rebase commet la dernière version de la succursale que vous fusionniez. Il y a beaucoup de conversations sur ce qui est meilleur si vous le faites ne recherche rapide sur programmeurs.se .

En ce qui concerne l'étiquette derrière elle, laisse traiter cela d'une perspective pratique. Lorsque vous traitez avec le nouveau code provenant de quelqu'un d'autre, il est toujours préférable de les amener à fusionner les dernières modifications de la succursale ou de la repasse fraîchement rebapsière avant de fusionner une fusion propre. N'oubliez-vous pas, ils ont écrit le code afin qu'ils soient généralement de loin les plus qualifiés pour traiter les conflits de fusion/rebase. Personnellement, je ne vois pas de problème avec cela et voir cette demande tout le temps des autres personnes. Pour moi, s'il n'y a pas de conflit, je le ferai souvent moi-même car il s'agit d'une deuxième mise à jour que GIT peut s'appliquer. Mais s'il y a des conflits, je demanderai toujours à l'auteur original du code de s'en occuper eux-mêmes.

De plus, pour GitHub (au minimum) spécifiquement, ils afficheront un lien vers votre fichier CONTRIBUTING au-dessus des tentatives de relations publiques afin que ce soit un bon endroit pour définir vos attentes et que de nombreux projets comprennent qu'elles ne feront que fusionner aux branches de jour.

16