2014-05-06 24 views
0

僅僅是爲了檢查代碼是否「正確」。你怎麼知道代碼是否真正運行,如果沒有接受並運行它就不會失敗。 (如果它錯了,那麼還原它)看起來有些複雜。測試應該由發起拉請求的人負責嗎?或者是否有一些工作流程在將代碼接收到主服務器之前運行代碼?拉/合併請求的要點是什麼?

回答

1

許多GitHub項目使用Travis (or other CI) integration在所有分支上運行構建,包括拉取請求。所以pull請求只是一個很好的git集成方式來提交補丁到一個項目。與普通的「錯誤報告」相比,我將它看作是「帶補丁的錯誤報告」選項。

+0

因此,如果您沒有CI,確保一切正常工作的責任取決於提交pull/merge請求的開發人員?實際的請求是更多的代碼風格檢查? – user2483724

+0

不,如果你沒有CI,那麼你的工作就是設置一個,如果你關心的事情繼續工作。 –

+0

因此,如果沒有配置項,合併請求永遠不應使用。在我的工作中,我們目前正在努力在整個公司中整合和促進CI。並不是說我們不認爲這是一個好主意,但它並不像「如果你在乎,只需設置它,否則你不在乎」這麼簡單。 – user2483724

0
  1. 審查代碼

    你要確保每個團隊成員都知道發生了什麼事或從代碼庫中取出一個項目時。此外,第二或第三......或第四或第五組眼睛不會受傷。

  2. 測試代碼

    一個CI服務(一拉TravisCI)會自動運行,以確保沒有被打破,是防禦的第一線,以確保綠色主構建 - 我不能足夠強調這一步。

  3. 討論代碼

    如果這種特徵/請求甚至是代碼庫的一部分?有更好的方法來實現它嗎?

  4. 檢查點

    當前部署是否破損?定位合併提交/拉取請求節點而不是單個提交節點要容易得多。

  5. 通信

    永遠不會有各地代碼足夠的溝通。有人總是在圈外