Hatena Blog Tags
はてなブログ トップ
Pull Request
このタグでブログを書く

Pull Request

このタグの解説についてこの解説文は、すでに終了したサービス「はてなキーワード」内で有志のユーザーが作成・編集した内容に基づいています。その正確性や網羅性をはてなが保証するものではありません。問題のある記述を発見した場合には、お問い合わせフォームよりご連絡ください。
stefafafan の fa は3つです•10 か月前

Pull Requestのdescriptionに「影響範囲」を書く理由

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

#チーム開発#GitHub#Pull Request

関連ブログ

なんでもノート•昨年

自分が出したPRのコードにコメントしすぎない

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

#Pull Request#コメント
every Engineering Blog•昨年

今すぐできるレビュワーに優しいPull Requestをつくる7つのポイント

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

#Pull Request#プルリクエスト#コードレビュー#チーム開発#GitHub
holyblueのブログ•昨年

1年前に書いた記事へのアクセス状況、世の中のRedmine/Gitの利用状況

1年前に、自分の覚え書きと、はてなBlog試し書きを兼ねて書いた記事「Windows環境のBitnamiでRedmine/Git連携(統合認証)」 blog.holyblue.jp を久しぶりに見てみたら、この1年間で250アクセスくらいありました。そのうち、今月だけで32アクセス。 アクセス元サイトの91%がBingからという結果。 どうやらBingではこの記事がヒットするみたい。 でもGoogleではさっぱりなのであろうということがわかりました(^^; Bingでの検索ワードを見てみると、 git redmine 連携 windows bitnami git redmine bitnami…

#git#Pull Request
アクトインディ開発者ブログ•4 年前

VSCodeにGithub純正のプルリクエスト拡張が来た！

morishitaです。 VSCode に Pull Request のコードレビューをサポートする拡張はいくつかありますが、ついに純正が出ました！ まだプレビュー版ですが、インストールして使えます。 VSCode のブログでも紹介されています。 code.visualstudio.com できること PR のリストの参照 PR 内の変更ファイルリストの参照 変更ファイルの参照とコメント登録（markdown サポート） ワンクリックでローカルにチェックアウトもできる。動作確認しやすい インストール 通常の拡張の様にインストールできます。 次のページのボタンを押すのが手っ取り早いでしょう。 m…

#VSCode#GitHub#Pull Request
TiDB commits•15 時間前

TiDB Commits on 2022-06-29

github.com ddl: DROP TABLE/VIEW/SEQUENCE now use XXXStmt as parameter by lance6716 · Pull Request #35741 · pingcap/tidb · GitHub DROP TABLE/VIEW SEQUENCEがast.DropTableStmtなどの引数を利用するようにしています。 metrics: add loading region cache duration by you06 · Pull Request #34679 · pingcap/tidb · GitHub loading reg…

Retty Tech Blog•昨日

Terraform によるマイクロサービス環境の構築

この記事は #Rettyマイクロサービス強化月間 の第5弾の記事です。 engineer.retty.me 前回は鈴木さんの「予約 API のマイクロサービス化と gRPC ゲートウェイの置き方」でした。 インフラチームの幸田です。 自分の番はまだまだ先だと思っていたら、もう月末で焦って記事を執筆しています。 今回の記事ではマイクロサービス基盤を新しく Terraform で構築し直した話をしようと思います。 マイクロサービス化のはじまり システム構成 抱えていた問題 マイクロサービス間での設定差異や変更漏れの発生 新規サービスの追加に時間がかかる 作業に対するレビューができない 課題解決に向…

Sakiのプログラミング学習ブログ•昨日

フィヨルドブートキャンプのシステム開発プラクティスを卒業したので、振返りをしました

はじめに 1月末〜システム開発のプラクティスに入り、先日自分が作った全てのPRがマージされたので、振返りを書きました。 途中、自作サービスのエレベーターピッチ&ペーパープロトタイプ作成や技術検証に取り組んだり、1月中旬〜3月中旬の2ヶ月間のみ、働いているパートのシフトを週4から週5に増やしたり、FBC内でゴールデンウィークに開催されたEverydayRailsという書籍の短期集中輪読会に参加したり、とこのプラクティス以外にも色んなことに取り組んでいたのもあり、想定していたよりも時間がかかってしまいました。 システム開発に取り組む中で、たくさんの学びや気づきがあったので、「最近月ごとの振返りも書…

TiDB commits•昨日

TiDB Commits on 2022-06-28

github.com *: only add default value for final aggregation to fix the aggregate push down (partition) union case by tiancaiamao · Pull Request #35443 · pingcap/tidb · GitHub static pruning mode gets wrong results · Issue #35295 · pingcap/tidb · GitHub の修正のため、実行計画の最後の集計でのみNULLを付与するようにしています。 readme: r…

すたいるのメモ帳ブログ•一昨日

OBS Studio の 未実装機能 について

「OBS Classic」には実装されていて「OBS Studio」に未実装の機能をメモしています。 「OBS Studio」への乗り換えで機能が実装されてるか気になる方はご覧ください。 ※内容は「OBS Studioの未実装機能など」のままです。(移転しました) 記事投稿日 2021年09月30日、最終投稿更新日 2022年05月28日 14時30分頃 ※記事に情報を掲載するにあたり細心の注意を払っておりますが、それらの情報の正確性、有用性、安全性について、また最新であるか、あるいはご利用の目的にそったものであるかについて保証出来ません。

なるようになるブログ•一昨日

rails commit log流し読み(2022/06/27)

2022/06/27分のコミットです。 CHANGELOGにのったコミットは以下の通りです。 activerecord/CHANGELOG.md Fix supporting timezone awareness for tsrange and tstzrange array columns. Upgrade sqlite3 to fix broken ext compilation Gemfile.lockの修正です。 sqlite3を1.4.4に更新しています。1.4.3だとnative libraryのビルドに失敗してしまうバグがある為。 Fix: incorrect value reg…

カンムテックブログ•3 日前

バンドルカードと Pool のカードが 3D セキュアに対応しました

バンドルカードの SRE をしている summerwind です。最近は A Philosophy of Software Design を読んでいます。 タイトルの通り、2022年6月21日からバンドルカードと Pool のカードが 3D セキュアに対応しました。バンドルカードではアプリですぐに発行可能なバーチャルカードを含む全てのカードで対応しているので、気軽により多くの加盟店での決済にご利用いただけるようになりました。 いつもはバンドルカードのインフラやセキュリティといった領域を担当しているのですが、3D セキュアの対応では久しぶりにバックエンドエンジニアとして自分もプロダクト開発に関わ…

きり丸の技術日記•3 日前

【GitHub Actions】npmのpackage.jsonをアップデートするPull Requestを作成する

package.jsonのライブラリバージョンを、現在推奨されているバージョンまでアップデートする方法を探していました。そんなに多用するものではないので、GitHub Actionsでなんとかできないか探したのがこちらの記事です。 環境 taichi/actions-package-update v0.5.0 npm-check-updates 14.1.1 GitHub Actions 2022/06/26時点 npm 2022/06/26時点 対応 実行するトリガは手動実行する前提で考えています。1-2ヵ月ごとに最新化したい場合は適宜、onを修正してください。 taichi/actions…

yigarashiのブログ•3 日前

User Agent文字列を使ったブラウザ判定の事例 2022年版

やむを得ず、User Agent文字列を使って特定のブラウザ向けにJavaScriptの処理を分岐する必要が生まれてしまったので、調査・検討のログを記事にまとめます。 基本的にはバッドプラクティスである ユーザーエージェント文字列を用いたブラウザーの判定 - HTTP | MDN まずはMDNがドキュメントを公開しているので読みましょう。要点は以下です。 基本的にUser Agent文字列に基づいて処理を出し分けるのはバッドプラクティス 多くのケースではUser Agent文字列を使うよりも良い手段がある 例えば特定の機能の実装状況に基づく分岐を行いたければそれを直接検出する それでもやむを得…

koluku's blog•3 日前

GitHub Actionsでactions/github-scriptからJSONのレスポンスを取得する

※Zennからの移植記事です GitHub ActionsでGitHubに関する操作をしたい場合にREST APIを叩くよりも楽に書けるactions/github-scriptがある。APIの叩き方はOktokit.jsとほぼ同じJavaScriptで書くことができる。 （ただ最近はGitHubホストランナーのUbuntuなどには ghがプリインストールされているので、こちらを用いたほうが読みやすいなと思ってる） 下記はdevelopブランチにpushが発生した場合にPRを作成する例。 name: create pull request on push on: push: branches:…

Copy/Cut/Paste/Hatena•3 日前

gomockで「呼ばれるべき」と指定した関数の非同期な実行を待ちたいのでテストヘルパーを書いてみた

gomockでデータベース操作などをモックすることはよくあると思います。 そのテスト関数内で「呼ばれるべき（EXPECT()）」と指定した関数に非同期に実行される挙動がある場合、テスト関数の書き方によっては実行終了までにその関数が実行されないことがあり、結果としてテストがエラーになってしまうことがあります。 そして、エラーになったりエラーにならなかったりしてくると、Flakyなテストの出来上がりです*1。 （「そもそもテストしやすいように〜すべきだ」というテスト容易性については一旦傍に置いておきます） ものすごい雑コードですが、例えば以下のような感じのテストです*2。 func TestAsy…

コポうぇぶろぐ•3 日前

【Unity】MidiJackをAndroidで動かしてみた（検証：2022.06.12）

Androidで動作するUnityアプリでMIDIキーボードの入力を取得する方法として、MidiJack（Keijiro神 作）を使った方法を紹介します。 こちらはプルリク（momo-the-monsterさん）にてAndroidの対応をしている方がいらっしゃいましたので、その導入について調査した備忘録になります。こちらのプルリクは master にマージされておりませんので、MidiJackとは非公式な手段となります。 そして結論といたしまして…、残念ながらAndroidでのUSE MIDI接続の動作は不安定でした。（BLEは大丈夫そう？）!!注意!! 今回の記事は、見方によってはちょっとデ…

スマレジエンジニアyushiのブログ•4 日前

【TODOリスト 第10回】Golangのバージョン更新

久しぶりにTODOリストを作っていきます。 yushi-dev.hatenablog.com 今回は、Golangのバージョンを更新します。 Golangのマイナーバージョン固定 久しぶりにプロジェクトをビルドしようとしたら、エラーが出てしまいました。 docker compose build --pull => ERROR [3/3] RUN go get -u github.com/go-sql-driver/mysql gith 0.3s ------ > [3/3] RUN go get -u github.com/go-sql-driver/mysql github.com/gin-…

なるようになるブログ•4 日前

rails commit log流し読み(2022/06/25)

2022/06/25分のコミットです。 CHANGELOGへの追加はありませんでした。 Document concerns about storage and column size when using encryption rails guideのActive Record Encryptionの修正です。 encryptionは Base64 encodingを行う為、カラムのサイズが元のデータより多くなる旨説明したImportant: About storage and column sizeセクションを追加しています。 Fix code cache namespacing for p…