Pull Request

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

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

仕事では顧客の環境に合わせてGitHub、Bitbucket、GitLabを使用することが多いのですが、 エンタープライズな世界ではクローズドな環境でソースを管理したいという顧客もそこそこいるわけで そういった顧客の時にはGitBucketというオープンソースのGitHubクローンをオンプレ環境に立てて使用しています。 gitbucket.github.io コードレビューはプルリクエストの中で行うので、どれぐらいの指摘件数なのか、レビュー指摘がどんな内容だったかなどなど 分析のための情報を収集するためにはプルリクエストからコメントを抜き出す必要があります。 GitHubではAPIが用意されて…

記事内容 不要なコミット削除方法 git resetをして、ローカル内容が消し飛んだ時に、リモートの内容を引っ張ってくる方法 上記2点でハマったので、忘備録を兼ねて記事にしました。 1. 不要なコミットの削除のやり方 pull requestに、関係のないファイル(他のPRでコミットされたファイル)がコミットされてしまった。 原因はブランチを切るところを間違えたことと推測される。 その時の対処法をまとめた。 1. brew install tigでtigをインストール 用途としてはgit logと同じだが、tigの方が見やすいので、インストールした。 （こっちの方が、git logより見やすい…

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

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

これは「Git、GitHubの使い方」の１１回目になります。 はじめから読みたい場合はこちらになります。 kuku81kuku81.hatenablog.com 全体を通して見たい場合はこちらになります。 kuku81kuku81.hatenablog.com 初心者、初級者におすすめの入門書は以下となります。 よかったら見てみてください。 kuku81kuku81.hatenablog.com それでは本題になります。 ここでは、「git push」まで終了した後のプルリクエストとマージの方法について説明します。 個人ベースでの活用を念頭にレビュー、レビュー依頼の部分は省略したシンプルな内容…