附上一張 pr 的片段
不確定對方為何要這樣改，有先詢問原因，但尚未得到回應
雖這裡只是改簡單的常數命名，但之前寫的東西幾乎都被刪掉了，看了 pr 才知道
像是對方可能覺得這樣寫比較好，所以就改成他覺得ok的方式
但好不好、適不適合可能每個人的想法不盡相同
這狀況最近經常遇到，所以想請問大家在跟別人合作專案上遇道類似狀況是怎麼解決呢？如果對方態度強硬該怎麼溝通比較好呢？
這算是第一次在非工作場合中跟別人合作做專案
平常工作上因大部分都是與資深同事和技術主管討論，所以不太會遇到這樣的情況
不知道過來人或是各位前輩有什麼想法？
如果大家有什麼建議也不吝指教
感謝
已邀請的邦友 {{ invite_list.length }}/5
改程式主要考慮兩點原則
1.改了之後「功能增加」
2.改了之後「bug減少」
至於其他都不用考慮
1.程式誰寫的
2.程式在什麼時間寫的、花了多少時間寫的
3.程式在什麼地方寫的
4.程式花了多少心血寫的
這算是第一次在非工作場合中跟別人正式合作做專案，而這問題在合作過程中不斷遇到
平常在工作上很少遇到這樣的問題
你平常在工作上有沒有改過別人寫好的程式碼？
如果有，一定有你的原因才會改別人寫好的程式碼
而你應該也不會每次修改都先找到原作者打招呼徵得同意之後再修改吧...
正常程序:
Commit 之後, 需由作者+團隊一人, 共同做 Code Review, 兩人都同意才能 PR.
完整程序如下:
https://birkhoffg.github.io/blog/posts/how-to-commit-and-code-review-on-github/
如果兩人無法取得相同意見, 則由這個專案的 Project Owner 出面仲裁後, 選擇一方的意見完成 PR.
沒做過 Code review 的 Branch 不可以被 Merge 回 Master.
如果不確定他改的理由，建議可以詢問本人，或許他的方法比較好，但在對方沒有說服你之前，都不要 approved ，至於如果對方不管你，直接 merge 的話，下次就不要合作了吧...
首先調整前後的哪個程式碼比較好，這個不好說，因為沒看到這些變數是怎麼被使用的
合作的問題，的確要先跟對方說一下會改動到什麼地方比較好，或是在 PR/MR 留言說這裡有改動到他人的程式碼，原因是...，雙方達成共識才做合併
感覺只是遊戲規則沒講好，
看要不要參考airbnb的JavaScript Style Guide
然後你們二方有疑慮，還可以上去找人評論
iThome鐵人賽
看影片追技術