附上一張對方 pr 的片段
不確定對方為何要這樣改,有先詢問原因,但尚未得到回應
雖然只是改簡單的常數命名,但之前寫的東西幾乎都被刪掉了,看了 pr 才知道
不知道大家在跟別人合作專案上會不會遇到類似問題?
像是對方可能覺得這樣寫比較好,所以就把他人寫的東西都改掉,改成他覺得ok的方式
但好不好、適不適合可能每個人的想法不盡相同 主要還是需要多溝通(?
如果沒有合理理由,沒經過溝通,可能會影響合作下來的感覺
畢竟這是大家共同的作品 心血~
這算是第一次在非工作場合中跟別人合作做專案,而這問題在合作過程中卻是不斷遇到
平常在工作上倒很少遇到這樣的事情
不知道過來人或是各位前輩有什麼想法?
如果大家有什麼建議也很歡迎提出
非常感謝
改程式主要考慮兩點原則
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
然後你們二方有疑慮,還可以上去找人評論