Pull Request

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

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

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

gitを活用してファイル編集をした後に、通常通り、 git add、git commit、git push、・・・ を行っていたのですが、プルリクエスト実施後に、以下のようなメッセージが発生しました。 「There isn’t anything to compare. master and update_branch are entirely different commit histories.」 masterと自分達の編集したupdate_branchは全く異なるコミット履歴をもつもので比較できないということです。 確かに今回はいろいろなところをいじりまくったためそうなるかとも思いましたが…

わたしはWeb系のエンジニアをやっていますが、普段触っているプロダクトたちはPull Requestのテンプレ*1として以下のような項目が記入されています。 ## 解決したい課題 ## やったこと ## 影響範囲 ## 確認方法 今回はこの「影響範囲」について何故書くと良いと思うのかについて言語化を試みます。 どういう変化が起きると想定しているのかが伝わる 新しい機能をこれからリリースするぞというPull Requestを作る際に、実際にそのPull Requestをマージしてリリースした瞬間からユーザにお披露目となるのか、それとも今はまだフィーチャートグルなどで隠しているからこのままリリース…

PR = プルリクエスト。GitHubのPRの話。 自分が出したPRのコードにコメントをすることがある。レビュアーがレビューしやすいように補足説明している。自分以外でもこのようなPRを見かける。 それ、PRではなくコードにコメントを残すべきでは？ と、ふと思った。 補足説明がないとレビュアーに意図が伝わらないのであれば、おそらく後で読んだときにわからなくなると思う。であればコードにコメントを残した方がよいと思う。 もちろんPRレビューのときにだけ必要な情報もあると思うので、すべてコードにコメントすべきということではない。使い過ぎに気を付けたい。

はじめに 今すぐできるレビュワーに優しいPull Requestをつくる7つのポイント 1. WhyとWhatをそれぞれ記載する 2. 説明文は構造化する 3. コミットは課題を解決した単位で行う 4. Pull Requestは適切な大きさに分割する 5. 個別説明が必要な箇所は積極的にコメントをつける 6. テストを書く 7. Pull Requestでのコメントを Slack に通知させる さいごに はじめに はじめまして。DELISH KITCHEN開発部の桝村です。DELISH KITCHENのWEBフロントやAPIサーバーの開発等に携わっています。 突然ですが、みなさんは本日もPu…

github.com *: fix and update OWNERS file by ti-chi-bot · Pull Request #44377 · pingcap/tidb · GitHub OWNERSファイルを更新しています。 github.com 2023年6月4日にPDのコミットはありませんでした。 github.com 2023年6月4日にTiKVのコミットはありませんでした。 github.com 2023年6月4日にdocsのコミットはありませんでした。

2023/06/03分のコミットです。 CHANGELOGへの追加はありませんでした。 Fix has_one autosaving for CPK associations activerecord/lib/active_record/autosave_association.rbの修正です。 composite primary keysを使用しているassociationをhas_oneで定義した場合に、通常のassociatinoと同様にautosaving処理が動作するよう修正しています。 Require missing models in ActiveRecord tests Act…

github.com *: add wait lock time in tidb_trx by jackysp · Pull Request #43105 · pingcap/tidb · GitHub tidb_trxにwait lock timeを追加しています。 ddl: enable linter revive by tangenta · Pull Request #44380 · pingcap/tidb · GitHub Reviveの設定を更新しています。 *: delete useless `TiKVMinVersion` param by Defined2014 · Pull…

複雑性の高い部分の探し方 まとめ Zennに「Flogを使用したリファクタリングの必要性の可視化と負債返却の説得の仕方の一例」を投稿しました。 zenn.dev 所感 カスタマイズが少し弱い部分があるのでPull Requestを出してみたりしたいですね。 rubocopの内部仕様ももう少し把握したいですが、Zeitwerkをもう少し使いこなせると静的解析でモジュラーモノリス化する際のドメインを機械的に導き出せる気もするの調べたいところ。 ランキング参加中プログラミング

2023/06/02分のコミットです。 CHANGELOGにのったコミットは以下の通りです。 activerecord/CHANGELOG.md Assign auto populated columns on Active Record record creation Remove the deprecation warning when prepared_statements configuration is set for the mysql2 adapter. Fix where on association with has_one/has_many polymorphic rela…

KanmuでPoolを開発しているhataです。最近、ロボット掃除機を買いました。ロボと猫がじゃれている景色はいいですね。 今回はGoのユニットテストの並行化についての記事です。 TL;DR Goのテストは、並行化することでテスト実行時間の短縮やテスト対象の脆弱性の発見などのメリットがある 基本的にはそのままでも最適化されているが、テストコードにt.parallelを記述することでよりきめ細やかな最適化を施すことができる ただし、一定規模以上のアプリケーションへの導入・運用は大変 テストコードを一気に並行化するtparagenというツールや、並行化忘れを防ぐ静的解析ツールがあり、これらを使うこ…

こんにちは。新規IAMプロダクトでフロントエンドアーキテクトを担当している@shisama_です。 この記事では、開発スピードを上げるためにGitHub Actionsを使ってプルリクエストごとにレビュー用の検証環境をGoogle Cloud Run（以下、Cloud Run）に構築する仕組みについて紹介します。 今回紹介する内容のサンプルのリポジトリはこちらになります。 github.com この仕組みは業務委託でお手伝いいただいていた@chimame_rtさんが考案し設計してくれました 👏 はじめに フロントエンドチームが抱えていた課題 プルリクエストごとに Cloud Run に検証環境…

2023/06/01分のコミットです。 CHANGELOGにのったコミットは以下の通りです。 activerecord/CHANGELOG.md Use the first key in the shards hash from connected_to for the default_shard. actionview/CHANGELOG.md simple_format helper now handles a :sanitize_options - any extra options you want appending to the sanitize. Set default_shar…

github.com linter: enable revive for part of planner by hawkingrei · Pull Request #44310 · pingcap/tidb · GitHub Linterの設定を更新しています。 *: refine log for `GenRLimit` by ywqzzy · Pull Request #44318 · pingcap/tidb · GitHub util.GenRLimitの引数を追加しています。 metrics: add metrics for avaiable RU by CabinfeverB · P…

エイプリルフールのいつものやつではありません。 株式会社はてなに入社しました - hitode909の日記 今年の3月に株式会社はてなにSREとして入社して3か月が経ったので、自身の振り返りも兼ねてブログを書いています。 SREのオンボーディングの雰囲気に関しても、このブログでお伝えできるといいかなと思っています。 自己紹介 id:taxintt です。 twitter.com 前職は株式会社ビザスクという会社で主にCore SREとして働いていて、はてなに転職してからは監視SaaSのMackerelの開発チ－ムでEmbedded SREとして働いています。 前職ではGCPを利用していて、AW…

はじめに こんにちは。FAANSバックエンドエンジニアの浜口（@xlgorbylx）です。普段はFAANSのバックエンドシステムの開発をしています。 FAANSとは、弊社が2022年8月に正式ローンチした、アパレル店舗のショップスタッフの販売サポートツールです。例えば、ZOZOTOWN上で実店舗の在庫取り置きができる機能や、コーディネート投稿の機能などを備えています。投稿されたコーディネートはZOZOTOWNやWEAR、Yahoo!ショッピング、ブランド様のECサイト等に連携が可能です。これによりお客様のコーディネート選びをサポートし、購買体験をより充実したものにします。機能の詳細に関しまして…

こんにちは @hsbt です。ゼルダの伝説 ティアーズ オブ ザ キングダムがついに発売されたので RubyKaigi 2023 の登壇後にプレイを開始していますが、引き続き原神もアップデートがあり、さらに崩壊：スターレイルも始めてしまったので大変なことになっています。 さて、今回は 5/11-13 に長野県松本市 まつもと市民芸術館 で開催された RubyKaigi 2023 で私 @hsbt が廊下会議で何を話してきたのか、という内容をご紹介します。 廊下会議とは 本日の技術顧問業、「技術カンファレンスへ参加するときに大事にしたいのは廊下（自然発生的に起こるインフォーマルなオフライン議論の…