Hatのエンジニアはどうしているのかプロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合はどうしたらよいだろうか。Hatのソフトウェアエンジニアが、コードレビューを行うに当たって念頭に置くべき10のヒントを解説した。});この記事は会員限定です。会員登録(無料)すると全てご覧いただけます。Hatでソフトウェアエンジニアを務めるデビッド・ロイド氏は2019年7月8日(米国時間)、コードレビューを行うに当たって念頭に置くべき10のヒントを同社の開発者向け公式ブログで解説した。プロジェクトメンテナーの立場にあるとき、提出されたコードが何らかの理由で気に入らない場合に役立つ指針だ。コントリビューター側としても参考になる。これらのヒントは、客観的で的を射たレビューを行い、プロジェクトとその参加者を前進させるという観点からまとめられている。良い例:「どうすれば、あなたの変更を受け入れるとともに『XXX』を行えるでしょうか」(2)大げさな表現を使わない望ましい結果になるように、簡潔に懸念事項を述べたり、質問したりする。良い例:「『X』を行うと、既存の『Y』が遅くなる可能性がありそうです。別の良い例:「この変更を行うと、シングルループ『O(n)』が二重にネストされたループ『O(n2)』になります。これはパフォーマンスに影響しないでしょうか」(3)悪意を表に出さない心に留めておいた方がよい意見もある。礼儀正しくできないのであれば、関与しない方がよい。悪い例:「こんなコードを書いても、ソフトウェアエンジニアがあなた自身に合ったキャリアパスだと確信できますか」(4)建設的に関わる問題解決の方法について、提出されたコードとは異なるアイデアがあったとしても、建設的に関われば、その2つの選択肢より、さらに良い解決策が見つかるかもしれない。悪い例:「この変更はお粗末ですね。良い例:「『XXX』の場所で似たような変更を施したことがあります。良い例:「ちょうど似たような変更を進めていますが、『ZZZ』という理由で『X』をすることを選びました。設定を有効にする必要があります。beta」を発表し、コードに対するコメントを処理する際の便利な機能を追加した。エディタ横のドックにレビューコメントを表示したり、コメント行をドックにdiff表示したりできる。クックパッドのモバイルアプリ開発が「機械に人が合わせる」リリースフローに行き着いた理由クックパッドのモバイルアプリ開発チームは「機械が人に合わせるのではなく、機械“に”人が合わせる」という新しいアプローチを採用した。この新しいリリースフローに至るまでの道筋と効果とともに、同社の茂呂智大氏が@ITソフトウェア品質向上セミナーの基調講演で明かした。Actions」は、一言で言えば「開発者のワークフローを自動化するサービス」。一般的な自動化ツールではなく、開発者がそのパワーを直接、自身の問題を解決するために構成し、利用し、共有できるものだという。メールマガジンをご購読ください。申し込みページへITmediaはアイティメディア株式会社の登録商標です。