2017-10-13 1 views
0

Notre équipe a récemment présenté Github Enterprise et j'ai eu des difficultés avec les PR.Les validations non liées sont incluses dans une requête pull sur l'entreprise Github

J'ai créé une branche 'funk/worker_item # digests' à partir de master puis je commets test A après avoir vérifié la branche puis l'avoir poussée. Quand j'ai essayé de créer un PR, j'ai remarqué que la branche avait aussi d'autres commits non liés, qui ont été mentionnés par de nombreuses branches comme illustré here. Qu'est-ce qui aurait pu causer ça?

Répondre

0

Un PR est une branche. Ce ne sont pas des commits «non liés», mais les ancêtres de la pointe. Pour fusionner un PR comprendra la pointe et tous ses ancêtres. Si vous ne voulez pas ces ancêtres, vous devez trouver la base de fusion de votre branche PR et la branche cible par git merge-base <PR-branch> <target-branch>. Créez une nouvelle branche à partir de cette base et appliquez la validation test-A dessus, par git cherry-pick ou git rebase ou d'autres méthodes. Et puis pousser cette nouvelle branche en tant que PR. Ce PR a seulement une nouvelle validation qui n'est pas encore appliquée dans la branche cible. Je ne suis pas familier avec le fonctionnement de PR dans Github. Peut-être que vous pouvez dire au propriétaire du référentiel d'appliquer uniquement la dernière validation de votre PR actuel par git cherry-pick, au lieu de fusionner l'ensemble du PR.

+0

Merci pour votre réponse! J'ai compris que test-A était le seul commit pour cette branche 'funk/worker_item # digests' quand j'ai créé la branche à partir de master puis l'ai vérifié et commit. Mais oui, test-A a un commit parent, auquel se réfèrent deux branches, 'funk/worker_item # digests', que je peux comprendre, et une autre branche 'funk_add_tag'. Il serait vraiment apprécié si vous me donnez des commentaires sur la raison pour laquelle cela se produit ... – uemurm

+0

@uemurm S'il vous plaît lire https://git-scm.com/book/en/v1/Git-Branching-What-a-Branch-Is et https://en.wikipedia.org/wiki/Directed_acyclic_graph. – ElpieKay