2017-02-23 114 views
0

當我的同事或我不同意某段代碼或某種方式在拉請求中執行某些操作時,我遇到了一些問題。我想知道你在評論員不同意編碼器的情況下如何解決討論。如何在github中討論拉請求

1)PR將不獲批准,並不會合併

2)編碼器不接受審查,並拒絕評論。無論如何,代碼將被合併。

上下文是審閱者和拉請求所有者與 開發者在同一個平面上,並且都是該項目的所有者。

感謝您的意見

回答

0

假設兩個人有相同的所有者權限,一個是PR的作者,另外一個是評審員:

  1. 測試

    一碼應該進行測試,如果它實際上解決了所需的問題。如果不是,那麼就不需要合併。

  2. 申請變更

    如果代碼解決問題或實現一個新的功能,那麼設計問題上升,如壓痕,標籤/空間,愚蠢地命名變量,甚至一個代碼模式/風格指導原資源庫可能會試圖保持接近。

  3. 代碼是罰款

    如果代碼兩側(ALGO +設計)的罰款,並仍有人誰不贊成,也有相當多的這些問題:

    • 代碼只是沒有按「噸覺得不對勁
    • 不必要的功能
    • 內部問題 - 也許審閱和PR的作者不喜歡對方

如果第三個「階段」出現,那麼在這種情況下,大多需要第三個人(具有相同級別的特權)來判斷代碼。

這將會是一種烏托邦式的思維,一個第三人總是解決這樣的情況,所以在這裏,你應該習慣了一些結局:

  • PR將保持打開狀態,解決在未來
  • PR將被關閉

    歡迎您就這個功能叉子和工作,這是在Git的性質畢竟(見DVCS),並可能提出的代碼放在一個新的PR在未來

  • 公關將合併
+0

我非常感謝您的意見,並有第三人稱這是一個好主意。感謝您的意見。我希望更多關於如何解決知識產權中的消沉和思維差異的經驗的評論 – Keniako

+0

@Keniako我不是那麼樂觀,因爲在我寫作的時候已經有3票贊成關閉這個問題,我可能是唯一一個充分關心實際上試圖回答這樣一個廣泛的問題,這可能甚至不直接與SO有關,而是SE的某些其他部分,例如[軟件工程](http://softwareengineering.stackexchange.com)。 :) – KeyWeeUsr

+0

我知道這是一個廣泛的問題,但這是一個事實,討論存在於日常工作中。我做了這個問題,因爲我的團隊通常會遇到這個問題,我需要聽到其他團隊如何解決這個問題。例如,以第三人爲例,這是一個簡單的想法,但這並不是我們想到的。我喜歡有更多的意見和想法。但是,如果這個話題對別人不感興趣,或者沒有這個問題,那對我來說就沒問題。 (即使有人投反對票,我也可以理解) – Keniako