Logo開発品質ガイドライン

変更内容確認

レビューには実装者1名、レビュアー1名以上の2名以上が関わります。
一般的にレビューはエンジニアが行いますが、当社では品質確保の観点から、UI変更がある場合はデザイナーもレビュー参加します。
レビュー時点で仕様実装不備を検出することにより、後工程でバグ検出するよりも工数の抑制が期待できます。

実装者が気をつけるべきこと#

プルリクエスト作成 を参考に、レビュースコープがレビュアーへ正しく伝わるよう配慮すること。

デザイナーによるレビュー#

デザイナーは仕様やデザイン観点でのレビューを行う。

  • 仕様通りの挙動となっているかをエビデンスで確認
  • デザイン観点の確認

また、デザイナーがレビュー参加することで以下の恩恵がある。

  • 実装着手後に仕様変更点があった場合などは、最悪レビュー時にキャッチアップできる
  • デザイナーが開発の進捗を把握できる
  • 作業者がデザイナーとコミュニケーションとれる

エンジニアによるレビュー#

エンジニアは、実装作業を仕様や保守の観点から総合的にレビューを行う。

また、修正箇所の指摘については 修正箇所の指摘 を参照すること。