コードレビュー - hitode909の日記
コードレビュー，慣れるとできるけど，いきなりdiffを渡されて，どうぞ見てくださいと言われてもよくわからないと思う． やりましょうというのはいいけど，ただむやみに読んでもうまくいかない．変更がある程度大きくなるとdiffだけ見てもよくわからないので，いろいろ見ることになる． 僕はいつも以下のようなことを無意識にやってて，うまくいってる気がしてる．GitHubのPull Requestの仕組みを使ってる前提で． Discussionをさらっと眺めてどういう問題を解決したいのか見る Commit Statusを見て，テスト通ってることを確認する Commitsタブで1コミットずつブラウザの新しいタブに開く 全部クリックし終わったら古い順に1コミットずつ読む 気になる点があったらエディタとかにメモしておく．あとで書き直されるかもしれないので，まだコメントしない 全コミット見終わったらFiles
Every Software has Bug(s) - steps to phantasien
Chromium プロジェクトは crbug.com というドメインをもっている. このドメインは Chromium のバグトラッカー (BTS) 専用の URL shortener として使われている. たとえば crbug.com/125981 にアクセスすると https://code.google.com/p/chromium/issues/detail?id=125981 にリダイレクトされる. URL shortener といっても t.co や bit.ly みたいに大げさなものではない. Apache…じゃなくてなぜか IIS だった…の設定か何かでちょこっと URL を書き換え, そこにリダイレクトするだけ. データベースなし. オンラインでバグの話をするとき, 多くの Chromium 関係者は crbug.com の URL でバグを表記する. バグの URL が短い
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
