Pull Request

頑張った。 とっても軽微な修正だったけど、PR上げる時は本当に緊張した。 何にPR出したの？ 出したのは二つ。 一つはAIアプリ開発プラットフォームのDify github.com もう一つはmrubyのライブラリ、httpクライアントのmruby-simplehttp github.com

はじめに セーフィー株式会社の AI Vision グループでテックリードを務めます橋本貴博です。 私たちのチームでは、レビューの属人化や特定のメンバーへの負担の集中が課題となっていました。どのメンバーがレビューするかによって、そのやり方やフィードバックの質にバラつきが出ることも少なくなく、これがコードの品質に影響を及ぼしていました。 そのため、チーム全体でレビューを行うことで、コードの知識を全員で共有し、レビューのやり方や品質を標準化したいと考えていました。こうすることで、誰がレビューしても同じように高品質なコードが保たれるようになります。 さらに、Pull Request (PR) の数を…

エンジニア・チーム開発経験が5年以上ありますが、未だに「コードレビューは難しい」と感じています。この理由について、自分の中で言語化出来てきたので、それについてまとめます。 コードレビューの効果 これらのメリットを最大限享受するのは難しい トレードオフを考えるのが難しい理由 コードレビューの指標が多岐に渡っている上に複雑 トレードオフのどのラインを狙ってコードレビューするべきかが明確になっていないことが多い プロジェクトによって重視する指標が異なる 人によってスキルが異なる 人によってコードレビューに関する心構えが異なる コードレビューの費用対効果を最大化させるためにできそうなこと レビュイー・…

こんにちは！もとひろです。 今回はGitHubでissueやプルリクエストを検索する方法を書きたいと思います。 はじめに 結論 検索方法 ① ドロップダウンメニューを使った検索の仕方 ② 検索窓を使った検索の仕方 タイトル、本文、またはコメントで検索 Issue やプルリクエストに関係したユーザで検索 プルリクエストレビューのステータスおよびレビュー担当者で検索 Issue やプルリクエストがクローズされた時期で検索 プルリクエストがマージされた時期で検索 他の検索方法 おわりに はじめに 最近gemのバージョンアップに伴う変更点等を調べたいとき、issueやプルリクエストがうまく調べられず苦…

大きなPull Requestのレビューがつらい 修正ファイル数が多いこと自体が問題なのではない 1つの内容に集中する 小さなPull Requestの作り方 リファクタリングの修正は気になっても別で出す Web API 1つに着目して実装を切り分ける 小さなPull Requestで作ったときのリリースの仕方 featureブランチを作って、そこから更にブランチを作っていく フィーチャートグルを使う 小さいPull Requestで小さくフィードバックをもらおう 大きなPull Requestのレビューがつらい 転職ドラフトでWebアプリケーションエンジニアをしている @iwtn です。 こ…

開発生産性 Advent Calendar 2022 16日目の記事です。 はじめに ペイトナー株式会社の脇田（@shimpeee_）です！『ペイトナー ファクタリング』開発チームでエンジニアリングマネージャー兼スクラムマスターとして、開発生産性と日々向き合っています。 「コード品質？レビュー効率？いや、PR数だ！！！」これは、他の誰でもなく、半年前の自分に声を大にして伝えたい叫びです。 「PR作成数をKPIにすると良い」とは知っていましたが、実は勘違いしていました。 コード品質やレビュー効率が改善された結果、PR作成数が増えると思っていました。ですが、実際は逆でした。 PR数を増やそうとする…

gitにてプルリクエスト実施後に発生したエラーとその対処方法についての内容です。 目次 １．発生したエラー内容 ２．今回解決した方法 １．発生したエラー内容 gitを活用してファイル編集をした後に、通常通り、 git add、git commit、git push、・・・ を行っていたのですが、プルリクエスト実施後に、以下のようなメッセージが発生しました。 「There isn’t anything to compare. master and update_branch are entirely different commit histories.」 masterと自分達の編集したupda…