「おまえは今までレビューしたプルリクの数をおぼえているのか？」 - pixiv inside

こんにちは、kanaです。社内ではpixivというサービスでPHPとTypeScriptとVim scriptを書く仕事をしています。今日はpixivの開発におけるコードレビューの話をします。 問題 pixivは昨年でサービス開始から10周年を迎えました。サービス開始当初と比較すると山のように新しい機能や画面が増えています。なのでpixivのコードベースは巨大です。PHPファイルだけでも5000個以上あります。 kana@pixiv ~/pixiv (2) [master] ^-^)/> git ls-files '*.php' | wc -l 5555 これだけの数のファイルを全て把握するのは無理です。なので、各々の開発者が自由にコードを書いて混沌にならないよう、設計方針を始めとして各種コーディング規約が整備されており、秩序を保っています。 とはいえ誰もが常に完璧なコードを書けるわけでは

[ShinSemiya]

進行中のレビューが一人に１０個も集中する状況が異常なのでは。その状況を改善せずに自分以外の人に投げさせるというのは本質的解決策ではなさそう。

[fumisan]

可視化w

[braitom]

レビュー依頼が飛んできているPRの数をSlackのStatusに反映させて可視化させる方法について。PRのURL付きのメンションもらったらカウントを増やしている。なるほどおもしろい使い方だ。

[maricar9710]

“実際に「自分は忙しいから他の人にレビューを依頼してね」オーラを出すことにも成功している” ∞つらそう

[shinjukukumin]

マネしたい。

[n_231]

pixivのphpファイル数は5555。ちぃ覚えた。

[luccafort]

ロードバランシング的にレビュー依頼を分散するのかと思ったらレビューを可視化する方向の話しだった。それはそれとしてよい。

[zonu_exe]

いま見たら*.phpのファイル数が増えて5565個になってたので消す活動やります。 ; でもこのファイル数はテストコードを含んでるので悪いことばかりじゃないよ

[honeybe]

いいな

[FromAtom]

コードレビューで消耗しなくなって便利だ。

[azumi_s]

pixivでの事例

[geta6]

slackステータスの活用方法、発見される

[ik11235]

"pixivのコードベースは巨大です。PHPファイルだけでも5000個以上あります。"

[bash0C7]

ナイスHackだ