努力踏出舒適圈®Div
@Paul_shu
Thu, Nov 15, 2018 7:06 AM
前面的工程師 排版歪七扭八的
每次我都要專門弄一個commit來排版很煩耶
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 7:26 AM
沒事沒事，現在我手上的 repo 共 208 個 commit，有 20 個就是 fix coding style
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 7:28 AM
公司的 repo 203/2143 跟 coding style 有關（而且大多是我 commit 的）
努力踏出舒適圈®Div
@Paul_shu
Thu, Nov 15, 2018 7:32 AM
還有一堆無用空格 今天在vscode 裝了 trailing space 然後公司專案一堆無用空格
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 7:35 AM
你可以試著在你公司推 CI 啊， commit 不合格就 push 不上去
努力踏出舒適圈®Div
@Paul_shu
Thu, Nov 15, 2018 7:38 AM
芥龍@移民海拉爾 : 其實我們公司之前有用gitlab ci 可是現在沒用了不知道為什麼
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 7:39 AM
八成是不確實（？
​蔥🐳
@knowlet
Thu, Nov 15, 2018 8:45 AM
芥龍@移民海拉爾 : 應該要 standard 沒過不能 commit 才對吧 XD
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 9:10 AM
​蔥🐳 : commit 在本地端，我是覺得不能要求每個人每個 commit 都符合 standard，但至少在推上去給大家看到的時候要符合標準
​蔥🐳
@knowlet
Thu, Nov 15, 2018 9:17 AM
所以你是 push 讓他上 feature branch 然後 CI 如果 standard 有過才會把他 merge 進去之類的？
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 9:20 AM
我們公司的策略是如果測試沒過（Standard、Unit、Feature）就不能 merge 進某個 branch
​蔥🐳
@knowlet
Thu, Nov 15, 2018 9:22 AM
芥龍@移民海拉爾 : 那如果是 standard 問題的話還不是要改，那在 pre-commit 的時候檢查還比較有效率也不浪費 worker 資源
芥龍@移民海拉爾
@song374561
說
Thu, Nov 15, 2018 9:42 AM
因為現在 standard 的檢查跟 unit test 是一起的，所以才沒做成 pre-commit
而且 pre-commit hook 是每個人要寫在自己的 .git/ 裡面，鑑於開發習慣不同所以沒有統一
載入新的回覆