こんにちは、kanaです。社内ではpixivというサービスでPHPとTypeScriptとVim scriptを書く仕事をしています。今日はpixivの開発におけるコードレビューの話をします。 問題 pixivは昨年でサービス開始から10周年を迎えました。サービス開始当初と比較すると山のように新しい機能や画面が増えています。なのでpixivのコードベースは巨大です。PHPファイルだけでも5000個以上あります。 kana@pixiv ~/pixiv (2) [master] ^-^)/> git ls-files '*.php' | wc -l 5555 これだけの数のファイルを全て把握するのは無理です。なので、各々の開発者が自由にコードを書いて混沌にならないよう、設計方針を始めとして各種コーディング規約が整備されており、秩序を保っています。 とはいえ誰もが常に完璧なコードを書けるわけでは
YAPCのスポンサーセッションで，DeNAの採用担当の方が話されていて，エンジニア文化への憧れから，コードの意味は分からないけど勝手にLGTMしたり，会場の発表スライドに載せるには不適切な画像を貼ったりしている，という発表をされていた． 単に迷惑そう，と思ったのと，それ以上に悲しくなって，自分たちが大切にしていることを軽んじられると悲しい気持ちになる． LGTMな画像を貼るのは，傍目から見ると，にぎやかな画像が出てきて楽しそうな雰囲気があるけど，画像を貼る前にはコードが正しいか検証しているのであって，ミスの許されないシリアスな場所でもある． 見た目がおもしろそうだからといって遊びに来られると迷惑だし，そういうライトな活動をするような，遊んでいるように思われていたのか，という悲しさがある． 逆に，そんなシリアスな活動をしているなら，そうと分かる真剣そうな雰囲気になっているべきという気もして，
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く