Est-il correct de demander aux contributeurs de rebaser leurs demandes de pull sur github

Je maintiens un dépôt github relativement populaire.

Lorsqu'une demande de tirage est bonne à fusionner, je demande généralement à l'auteur de la rebaser en un seul commit avant de la fusionner (en particulier lorsqu'il y a eu plusieurs petites modifications).

Est-ce une bonne pratique git? Cette étiquette GitHub est-elle acceptable / standard?

Donc, quelques avantages:

• J'obtiens un joli historique de commit propre dans les logs de commit
• Je n'ai pas besoin de modifier le commit 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.

En ce qui concerne Git, c'est une sorte de guerre sainte, que vous deviez simplement fusionner les branches ou rebaser les commits sur la dernière version de la branche dans laquelle vous fusionnez. Il y a beaucoup de conversations sur ce qui est mieux si vous effectuez une recherche rapide sur Programmers.SE .

Quant à l'étiquette derrière cela, permet de traiter cela d'un point de vue pratique. Lorsque vous traitez un nouveau code provenant de quelqu'un d'autre, il est toujours préférable de le faire fusionner les dernières modifications de la branche ou le rebaser avant de le fusionner pour garantir une fusion nette. N'oubliez pas qu'ils ont écrit le code, ils sont donc généralement de loin les plus qualifiés pour gérer les conflits de fusion / rebase. Personnellement, je ne vois pas de problème avec cela, et je vois cette demande tout le temps d'autres personnes. Pour moi, s'il n'y a pas de conflits, je le ferai souvent moi-même car c'est une mise à jour de deux secondes que git peut appliquer lui-même. Mais s'il y a des conflits, je demanderai toujours à l'auteur d'origine du code de s'en occuper lui-même.

En outre, pour GitHub (au minimum) en particulier, ils afficheront un lien vers votre CONTRIBUTINGfichier au-dessus de toutes les tentatives de relations publiques, ce qui constitue un bon endroit pour décrire vos attentes et de nombreux projets incluent qu'ils ne fusionneront que les branches à jour.