コードレビュー ありがちな問題への対処例 - Crieit
コードレビュー、これまでいろんなプロジェクトで経験して、意外と使われていないノウハウがあったり、風習が違ってつらみがあったりしたので、いろいろまとめてみる。 指摘事項について よくある話 - 駄目コードを憎んで人を憎まず。駄目なのはコードであって人格じゃない - 指摘する人は人格攻撃せずにコードのどこが悪いのかを指摘しましょう - 指摘される人も、言われているのはコードの問題であって人間の問題じゃないので、素直な心で受け止めよう この辺はみんな知ってると思うので略。ぼくが思う大事なルール コードレビューで指摘された内容は、対応必須ではない 理由： 対応必須にすると、「これ言ったらリリースできなくなるよね」みたいな忖度が発生してコメントできない人が出現するから。 絶対ダメとは言わないけど、あまりよくはない、みたいな指摘については、そのときは急ぐからリリースするけど、次回から気をつけるとかがあ
改善を続けるSonicGardenの中途採用におけるRailsの技術力の教育プロセスをご紹介
このエントリは、ソニックガーデン Advent Calendar 2015 の18日目の記事です。 現在、SonicGardenでは、中途採用のプロセスで「トライアウト」というWebシステムを自分たちで開発しながら、応募者とのやりとりを進めています。 そのやりとりの中では、応募者の方の人となりを伝えていただいたり、SonicGardenでのRails開発における技術力を身につけてもらっています。 ここでは、トライアウト中のRailsの開発力を身につけてもらうためにやっていることを紹介しようと思います。最後に、トライアウトを越えてきた人にとても評価の高いコードレビュー動画をチョットだししているので、ぜひ見てみてください。 トライアウトの技術の学習プロセスは以下の流れになっています。 チェックリスト＋課題 必要な技術を一覧化して、課題を通して基本を学んでもらうコードレビュー動画 他の人が過去に
1
リリース、障害情報などのサービスのお知らせ
最新の人気エントリーの配信
処理を実行中です
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
レビューのアウトプットは、レビューアのレベルを超えない（まとめ）: 柴田 芳樹 (Yoshiki Shibata)
