2014-04-11 27 views
13

Ekibimiz, sürdürdüğümüz uygulamayı kod incelemesi için dahili olarak istek istekleri kullanıyor, ancak bir taahhütte bulunma zamanı geldiğinde, doğrudan doğruya master'a yönlendiriyoruz. Biz hepimiz repo'luyuz, ve tarafından kod birleştirmek için çekme isteğini kullanarak, birleştirme taahhütleri ile taahhüt tarihimizi kirletmekten kaçınıyoruz. Uygulama ekibimize dahil olduğundan, başka kimse etkilenmez.GitHub çekme isteği nasıl bir taahhüt mesajıyla kapatılır?

Bilmek istediğim, taahhüt iletimi "closes PR #30" gibi bir şeyle değiştirebilir miyim ve github benim için çekme isteğini otomatik olarak kapatır mı?

Sorunları kapatmak için taahhüt mesajları kullanabileceğinizi biliyorum, bu yüzden çekme istekleri için benzer bir şey olduğunu umuyorum. İnternetten ve SO'dan hızlı bir arama yaptım ve hiçbir şey görmedim.

+1

[Bu] (https://help.github.com/articles/closing-issues-via-commit-messages) Yardım sayfasından GitHub size yardımcı olabilir. – ZeissS

+3

Teşekkürler @ZeissS, ancak bu bağlantı sadece * sorunları * kapatmayı ele alıyor, istekleri çekmiyor. –

cevap

10

Muhtemelen bu GitHub help page'a bakmak istersiniz. Şöyle ki:

mesaj taahhüt yoluyla

Sen bir sorunu kapatmak için aşağıdaki anahtar kelimelerden herhangi birini kullanabilirsiniz:

  • yakın
  • düzeltme
  • düzeltmeleri
  • kapalı kapatır
  • düzeltildi
  • resol ettik
  • veya "Çözülmüş # 456" çalışacak Yani

çözüldü "# ​​123 düzeltir" giderir. Tüm çekme istekleri sorun olarak eşlenir, bu nedenle çekme istekleri için de çalışır.

Not: Çekme isteğini değiştirdiğinizden, birleştirilmemiş işlemlerle ilgili bir ileti görürsünüz. Böylece çekme talebine bakarak, PR mesajının (sadece düz kapalıya karşı) dahil edilmesinin mümkün olmayacağı açıkça görülecektir. Bu durumda, taahhüt mesajının ilk satırında anlamlı bir şey koymadığınız sürece, mesajı çekme isteğinde görebilirsiniz.

+6

"Tüm çekme istekleri, sorunlar olarak eşlenir, bu nedenle çekme istekleri için de çalışır." - Ahhh, * bu, eksik olduğum kısım! Bunu nasıl biliyorsun? Github bunu bağlandığınız sayfaya kısa bir cümle ekinde eklemiş olsaydı, soruma cevap verecekti! –

+1

Bunu test ettim ve işe yaradı! "Kararsız işlemler" uyarısı hakkında haklısınız, ancak PR'm de "skingsland bunu <2 dakika önce

+0

@SteveK Ne yazık ki, sadece bir süredir GitHub kullanıyorum ve çekme taleplerinin sorun listesinde olduğunu bildiğim için biliyorum. Katılıyorum, GitHub'un dikkat çekici bir yerde bahsetmesi yararlı olur. :-) – jszakmeister

İlgili konular