Pull Request

仕事では顧客の環境に合わせて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より見やすい…

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

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

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

2023/11/29分のコミットです。 CHANGELOGにのったコミットは以下の通りです。 activerecord/CHANGELOG.md When using a DATABASE_URL, allow for a configuration to map the protocol in the URL to a specific database adapter. Add some behaviour to ActiveSupport::InheritableOptions to make it quack more like a hash activesupport/lib/act…

今日はActiveJob&SidekiqがJobの情報をどんな感じでRedisに入れているのかを調べたりしていた。キューは queue:<queue_name> のキーで type list で格納している。RetrySetとかScheduledSetはまた別のキーになっていた。具体的なデータを見ると理解が一気に捗る。 AWS負荷試験入門を読んで、仕事で困っている問題の糸口を掴んだような気がした。早く検証したい。この本を読んでから負荷試験を行えば良かったなぁ。やったことがないことにトライするときには予習しよう。 軽く夕ご飯を作った。えのきとカニカマのナムル的なおかずを作って、なかなか美味しかっ…

はじめに こんにちは、CTO室アシスタントのちいです。 私の住む北海道は遂に雪が降り、雪遊びのワクワクと同時に 路面凍結と除雪に震える日々が刻一刻と近づいてきました。 さて、hacomonoでは11月に1ヶ月間、期間限定でCodeRabbit🐰を試験的に導入中です。 現在は36名の開発メンバーがアサインされています。 今回は開発メンバーからのCodeRabbitを使用した感想をご紹介します。 ※感想のまとめはChatGPTにお願いしました。 CodeRabbitを使った社員の声：オープンAI APIを活用するレビューコメント GitHub Actions上で動作するCodeRabbitは、Op…

どうも、レコメンド商品のシステム開発をしている野川と申します。 私は、2021年にモノタロウに新卒入社し、2022年5月からレコメンド商品の開発に関わり始めました。 モノタロウのレコメンド商品は、下の図の①～④の流れでクライアントサイドで表示しています。大部分の処理はJavaScriptで構成しており、UIもそのHTML部分をjQuery(JavaScript)で作成しています。 図：レコメンド商品表の流れ 入社当時私は、ソフトウェアエンジニアとして、「可読性の低いコードは駆逐するべきだ」「読みやすいコードだけが正義である」「理解しやすいシステムだけが皆を幸せにする」と心の底から考えていました…

2023/11/27分のコミットです。 CHANGELOGへの追加はありませんでした。 Remove unused DeprecatedConstantAccessor include actionview/lib/action_view/template/resolver.rbの修正です。 使用していないActiveSupport::Deprecation::DeprecatedConstantAccessorのincludeを削除しています。 Merge pull request #50170 from y-yagi/fix-db_system_change-to-trilogy rail…

このブログポストは『SIerとSESで働くのを完走した感想』を編集してWeb再録したものです。 rimarimadan.hatenablog.com 転職するときのとりとめもない感想 今の職場で（意図せず）4社目なので…。1 転職するときに強く感じたのは、「単純なエンジニアの数が足りないのではなく、業務経験がある程度あって自律的によしなにやってくれるエンジニアが足りない」という事実です。2つまり、業務経験がなければ「みんなが行きたがる形態の職場」には箸にも棒にもかからないのです。 未経験で転職市場に出て人材紹介系に行けば、だいたい紹介されるのはSES企業ではないでしょうか。転職活動時は自分もそ…

Terraform Cloud の「Run Tasks 機能」を使うと Terraform Cloud のワークフローと外部サービスと統合できて，プラン取得前・プラン取得後などのタイミングにセキュリティスキャンなど「任意の処理」を実行できる💡 例えば取得したプラン結果を外部サービスに送信して，何かしらの非準拠が発見された場合に terraform apply の前に止めるというワークフローを実現できるようになる❗️以下の図を見るとわかるはず〜 ＼( 'ω')／ Terraform Cloud Run Tasks Integrations Setup | Terraform | HashiCor…

こんにちは。SRE部の巣立（@ksudate）です。 我々のチームでは、AWS上で多数のマイクロサービスを構築・運用しています。マイクロサービスが増えるにつれて、CI/CDの長期化やリリース手法の分散など様々な課題に直面しました。 本記事では、それらの課題をどのように解決したのかを紹介します。 目次 目次 はじめに CI/CDのこれまで Release PRによるリリース CI/CD実行時間の長期化 マイクロサービスごとのリリースが難しい リリーサーの制限ができない ドメイン単位の並行リリース リリース手法が分散する ブランチ間の同期が必要 パイプラインの増加 CI/CD実行時間の長期化 リリ…

こんにちは、技術本部 Eight Engineering Unit でエンジニアをやっている茂木です。 普段はSansan株式会社で正社員としてフルコミットしつつ、情報系の大学に通っています。 今回は Reviewdog x RuboCop を使って、ローカル環境で Lint をかける機構を Eight に導入した話をしていきます。

プロフィールページを作成していきます。 現在はNuxtのバージョンを2から3に移行しています。 プロフィールページ カテゴリーの記事一覧 - スマレジエンジニアyushiのブログ 技術スタックは下記の通りです。 Nuxt.js SCSS Tailwind CSS 今回はファイル移行していきます。 ファイルのコピー まずは単にファイルをコピーします。 components, assets, pagesのファイルは単に新しいv3のプロジェクトに移動します。 静的ファイル（ビルド等が不要で、そのままブラウザに返すファイル）について、Nuxt2ではstaticというファイルに格納していました。 Nux…

tl;dr; タイトルが全て tl;dr; 今年の準備 リポジトリ PRとDatadogを見ながら当日やったことを振り返り 当日10:00〜18:00のDatadog晒し sinatraのAPM サーバ全台 puma MySQL 具体的にやったこと CloudFormationから起動した直後のインスタンスにベンチマーカーを実行（スコア：4148） いつものスニペットをコピーして参照実装をGoからRubyに変更（スコア: 3781） Sentryとddtrace（DatadogのAPM）を有効化（スコア：3497） rubocop-isucon導入（スコア：3389） GET /api/use…

Gatsbyで作っているブログサイトで、Tagの数が多くなってきたのでTagをまとめたカテゴリ単位の表示をさせたい JSONファイルに適当なマッピングを持たせてそれを参照させたいというようなケースが発生したのでやってみた 普通にJSONを読み込んでも良いが、せっかくGatsbyで作っているのでJSONからGatsbyのNodeとして扱いGraphQLからクエリできるようにしてみる 今回はgatsby-transformer-jsonを使う install yarn add gatsby-transformer-json 設定 const plugins: GatsbyConfig["plugi…