- 名前：高山
- twitter：@edvakf
- お仕事：
- pixivのGrowthに関する開発
- MySQLのクエリのレビュー
- 開発プロセス効率化
趣味ではWebGLでMikuMikuDanceビューワーを作ったりしました
http://d.hatena.ne.jp/edvakf/20111101/1320127019
コードレビューしてますか？
- タブ vs スペース
- 2スペース vs 4スペース
- 行末のスペース
- いちいち指摘するのも面倒
- エディタで目立つようにしてない人は何度も繰り返す
- 宗教戦争になることがある
- 別にどれが優れているとかではなくて、どれかに統一されてれば十分
- 人間が気をつける必要ない（機械的に直ってくれれば良い）
- PHP-CS-Fixerというコーディングスタイル自動修正ツールを使用
- いきなり55万行の .php 全部にかけるのは怖かった
- Merge Request を作った段階でコーディングスタイル修正コミットを付けてくれるようにした
- 最初 Ruby で作ったが、gitlab APIのgemよりPHPのライブラリのほうが出来が良かったので作り直し
- 開いている Merge Request だけを取得することが出来ない（githubはできる）
- コミットを作れるが、1ファイルしか編集できない（githubでも同じ）
- 1コミットのdiffは取れるがブランチ間のdiffなどは取れない（githubはできる）
→ gitlab API縛りで開発していたが、つらすぎたので生のgitコマンドを呼び出すようにした
問答無用でコーディングスタイル修正コミットを付けられるとうざかったので、Merge Requestを作った人に対してコーディングスタイルを修正したコミットをMerge Requestしてくれるように変更
Merge Requestにコミットが伸びた場合でも、常に最新のコミットに対して1コミットだけを追加してforce pushするように
- Merge Requestをくれるgitlab botを作った
- gitlab APIはけっこう足りてない
- github APIのはgitlab APIに比べると大分強力
- ただし今回のbot程度のものでも結局gitを生で使うことになりそう
- require漏れとか教えてくれるといいかも?
ネタをくれた@onkさん、ありがとうございました