nkazukiのブックマーク / 2014年4月26日

nkazuki id:nkazuki

2014年4月26日のブックマーク (2件)

Clean up by hamukazu · Pull Request #4 · OpenTouryoProject/OpenTouryo
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
nkazuki 2014/04/26
気持ち悪っ。元のプログラムのコメントはたしかにごみくずだが、今後大して関わる意思も無いのに半端に引っ掻き回すようなプルリク出すのはオナニー。議論ならはてブか自分のblogでやれ。
リンク
オープンソース - hitode909の日記
オープンソースというかGitHubでPull Requestベースで開発してると，Pull Request送るのは，不具合修正か機能追加か効率化のためが主で，リファクタリングのためにわざわざPull Request送ることは稀だと思った．その結果，20人以上がコミットする3000行のクラスが出来上がったりする．プロジェクトのオーナーはどこが悪いか知ってから直せるけど，そうでもない人は，動く限りは，どう実装されてるか気にならないし，開発に参加しようとは思わないのかもしれない．リファクタリングすることで，他人のPull Requestがマージできなくなるかもしれない，という遠慮があるのかもしれない． photo by Najwa Marafie - Free Photographer
nkazuki 2014/04/26
OSSのソース読んでるとtypoとか未使用変数とか結構見つけちゃうんだけど、リファクタリングのためだけにpull req送るべきか毎回悩む（送らないことのほうが多い
リンク
- 2014年4月30日
- 2014年4月26日
- 2014年4月25日