ひびのログ

日々ではないけどログを出力していくブログ

コードレビュー

コードレビュー観点

雑に考えてみた。 フロントエンドのコードレビューを前提としているが、観点自体はそれにとどまらないはず。 かっこ内には代表的な概念を書いた。 大観点 コード 設計 機能要件 非機能要件 テキスト スコープ 中観点の例 コード 読みやすいか 正しく書かれて…

コードレビューの複雑度を数値化しようとしている

コードレビューをするときに、「これは明らかに一回のコードレビューで見られる量じゃない」という PR が飛んでくることがある。 いい感じに評価できないかと思って、いい感じの計算式を作ってみた。 PR の複雑度を減らしたほうがいい理由 レビュワーの負担…

既存プロジェクトに Prettier を導入したけど、コードレビューが辛すぎる件(ラノベタイトル風)

お気持ち表明(いわゆるポエム)です。 前置き 現在関わっている JavaScript プロジェクトには Prettier が入っていません。 また、ESLint は入っていますが(Standard)、--fix をしていない状況です。 そして、Vue.js を利用してしますが、eslint-plugin-v…