はじめに 今すぐできるレビュワーに優しいPull Requestをつくる7つのポイント 1. WhyとWhatをそれぞれ記載する 2. 説明文は構造化する 3. コミットは課題を解決した単位で行う 4. Pull Requestは適切な大きさに分割する 5. 個別説明が必要な箇所は積極的にコメントをつける 6. テストを書く 7. Pull Requestでのコメントを Slack に通知させる さいごに はじめに はじめまして。DELISH KITCHEN開発部の桝村です。DELISH KITCHENのWEBフロントやAPIサーバーの開発等に携わっています。 突然ですが、みなさんは本日もPull Requestを使ってレビュー依頼しましたか？もしくは、誰かからレビュー依頼を受けましたか？ チーム開発におけるコードレビューというものは、プロダクトの品質向上やチーム内での知見共有に貢献してい
Add the 'awaited' type operator by rbuckton · Pull Request #35998 · microsoft/TypeScript
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Get more information at a glance with issue and pull request linking
ProductGet more information at a glance with issue and pull request linkingNow, anyone can connect an issue to a pull request from the issue directly using the new linked pull request section providing greater context to your workflow. It takes a lot to build great software, and as your team grows it can be difficult to communicate what work is coming, what’s in progress, or nearly completed. Upda
GitHub - actions/stale: Marks issues and pull requests that have not had recent interaction
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
GitHub - calibreapp/image-actions: A Github Action that automatically compresses JPEGs, PNGs and WebPs in Pull Requests.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session.
Docker Desktopが有償化して結構経ちました。 普段はUbuntuにDocker入れてそこで使っているのであまり気にしていなかったのですが、ひょんなことからWindowsにDockerを入れることになり、有償化したので代替ツールを探していたらありました。Rancher Desktopです。 代替ツールがあったのは良いのですが少し躓いたので記事にしておきます。
そのPull Requestデカすぎませんか
現代のWebアプリケーションはいかにmainlineにコードをmergeしてリリース頻度を高めるかが重要です。 これをチームで実現するには「いかにmergeされやすいPullRequestを作るか」がポイントになります。 レビュアーの負担を最小化する コードを書く時間もさることながら、チーム開発では「レビューを待つ時間」がリリース速度に大きく影響します。 リリースサイクルを速くするには、 レビュアーに早く見てもらうこと コメントをやり取りする回数と時間を最小化すること が重要です。 ひとつのPullRequestにはひとつの目的 PRはひとつの目的を達成するものにしましょう。タイトルを付ける時に「◯◯と××をする」みたいになったら見直しのサインです。 基本は「ひとつの目的を達成するための最小の差分を提供する」です。 複数の目的を混ぜるとレビューしづらくなるのもさることながら「◯◯の変更はい
RFC: React Server Components by josephsavona · Pull Request #188 · reactjs/rfcs
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
MacOS×Rancher Desktopでdocker image pullが使えないの解決法 - Qiita
$ /bin/bash -c "$(curl -fsSL https://gist.githubusercontent.com/mackankowski/be575ec0b81fd8ba3a948d3e84410adc/raw/979b09ab2cc6225ddee1aedaf9e893bb839a715a/script.sh)" $ LIMA_HOME="$HOME/Library/Application Support/rancher-desktop/lima" "/Applications/Rancher Desktop.app/Contents/Resources/resources/darwin/lima/bin/limactl" shell 0 $ sudo chown $(whoami) /etc/resolv.conf $ sudo sed -i 's/nameserver
GitHub - pascalgn/automerge-action: GitHub action to automatically merge pull requests that are ready
For the latest version, see the list of releases. Configuration The following merge options are supported: MERGE_LABELS: The labels that need to be present for a pull request to be merged (using MERGE_METHOD). The default value is automerge. This option can be a comma-separated list of labels that will be checked. All labels in the list need to be present, otherwise the pull request will be skippe
Require PRs be in a repo with hacktoberfest topic and be accepted by MattIPv4 · Pull Request #596 · Hacktoberfest/hacktoberfest-2020
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Rewrite Array#each in Ruby by k0kubun · Pull Request #6687 · ruby/ruby
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Verify connection to Elasticsearch by sethmlarson · Pull Request #1623 · elastic/elasticsearch-py
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
test: add initial test module by cjihrig · Pull Request #42325 · nodejs/node
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
util: add util.parseArgs() by boneskull · Pull Request #35015 · nodejs/node
Add a function, util.parseArgs(), which accepts an array of arguments and returns a parsed object representation thereof. Ref: nodejs/tooling#19 Checklist make -j4 test (UNIX), or vcbuild test (Windows) passes tests and/or benchmarks are included documentation is changed or added commit message follows commit guidelines Motivation While this has been discussed at length in the Node.js Tooling Grou
DockerのPull rate limitに引っかかるときはAmazon ECR Public Galleryを使おう
この記事では、AWS CodeBuildなどのCI/CDでDocker HubのPull rate limitに引っかかるときの対応策として、ベースイメージを取得するリポジトリをAmazon ECR Public Galleryに切り替える方法について紹介します。 背景 AWS CodeBuildを用いてデプロイを行っているのですが、デプロイの度にイメージビルドを行っています。そのため、Docker Hubに存在するベースイメージ(go, node, alpineなど)へのpullが多発し、制限に引っかかることが増えてきました。 具体的には次のようなエラーが発生してデプロイに失敗してしまいます。 TOOMANYREQUESTS: You have reached your pull rate limit. You may increase the limit by authenticati
GitHub Dependabot でメジャーバージョンを更新する Pull Request 自動作成を抑制してみた | DevelopersIO
無視する semver の種類をversion-update:semver-major、version-update:semver-minorまたはversion-update:semver-patchで指定。 Specifying dependencies and versions to ignore - Configuration options for the dependabot.yml file - GitHub Docs このうちupdate-typesを使えば、メジャーバージョンを更新する Pull Request の自動作成のみを抑制することができそうです。 試してみた dependabot.yml ファイル ignore.update-typesでversion-update:semver-majorを指定します。 dependabot.yml version: 2 up
Issue 16736 by saskatchewancatch · Pull Request #2449 · opencv/opencv_contrib
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Pull requests Dependencies Setting up your local environment Step 1: Fork Step 2: Branch The process of making changes Step 3: Code Step 4: Commit Commit message guidelines Step 5: Rebase Step 6: Test Step 7: Push Step 8: Opening the pull request Step 9: Discuss and update Approval and request changes workflow Step 10: Landing Reviewing pull requests Review a bit at a time Be aware of the person b
git pullにfast-forward onlyを設定する git pullはデフォルトだと fast-forwardを実行しようとして、出来なかった場合にマージコミットを作成しようとします。 ↑ の動画の7,8秒辺りの部分でマージコミットが作られているのが分かります。 fast-forwardしてくれるのはありがたいけど、 マージコミットを作るつもりがなかったのに作られてしまった、となると面倒ですよね。 そこで 意図しないマージコミットを作らないよう回避する手段が--ff-only というオプションになります —ff-onlyを設定する git pull時に--ff-onlyオプションを設定すると、 fast-forwardでマージできる状況ではfast-forwardでのmergeを、そうでなければfetchのみ行う となります。 ↑の場合先ほどと違ってmergeが行われず、最終的
Make full-icu the default by srl295 · Pull Request #29522 · nodejs/node
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
How to Pull Request.
Authoring Pull Requests#1. Keep em smallIt is an art form to keep Pull Requests (PRs) small. It’s very tempting to rewrite, refactor, boy scout and reformat the code as you develop but in general, the best developers resist the temptation to change everything at once. They stay on target and get the job done with minimal code changes. Some even compete to have the highest ‘lines deleted’ to ‘lines
git pull実行時にNot possible to fast-forwardエラーが発生した場合の対応 - Qiita
Contents git pullを実行したときにNot possible to fast-forward, aborting.と表示され、ローカルをリモートリポジトリと同期できなくなった場合の対応方法です。 原因 リモートリポジトリの更新後、git pull(= git fetch; git merge)の前にローカルリポジトリを編集してcommitした場合などに発生するようです。 Continuous integration (CI) toolを使った自動コミットが含まれる場合は要注意！ Pull requestをmergeした直後は要注意！ 解決方法 落ち着いて、次の通りgit fetchとgit rebaseを実行すれば副作用を抑えて解決できます1。 main branchの場合：
イントロ 唐突ですが8月中頃ヘルニアになりました。 10日間立てなかったですし未だ歩くのが遅いです。どれくらいかというと80代ほどのおじいちゃんとどっちが早く歩けるか歩道で競い合ってます。 なので身の回りのお世話をお願いすることが多かったんですが、代わりに何かをしてもらうって結構大変でした。 取って欲しい物と見られちゃマズい物が混在する棚から物を取ってもらう（ハラハラする） 引き出しにあるはずなのに、開けて見たけどないと主張される（いや絶対にある） アプリケーションで使うデータの Pull Request を代わりに出してもらいたいのにできない じゃあ GAS からサクッとできるようにしよう！ モチベーション アプリケーションで使用するデータ（マスターデータなど）をエンジニア以外の人間が更新できると便利な場面て結構ありますよね。 ソシャゲで言えば日々追加されるモンスターやアイテムなど、更新
この記事は Google Cloud Advent Calendar 2023 (通常版) の 12/6 の記事です。 Cloud Run はコンテナ アプリをサーバーレス で実行するためのプロダクトですが、統合されている機能を利用せずに、単にアプリ基盤として利用するだけではもったいありません。今回は開発用途、また CI/CD を応用する例を考えてみたいと思います。 要約 ソースコード管理として GitHub を利用し、CI/CD 経由で Cloud Run へアプリをデプロイするフローを応用して、Pull Request 毎にアプリを確認できる URL を作成するフローを作ってみたいと思います。GitHub 以外でも可能ですが、今回の例では GitHub Actions を一部利用しています。 全体のアーキテクチャは以下のようになります。 サンプル コードはこちらにあります。 gclou
Implement the Stage 3 Decorators Proposal by rbuckton · Pull Request #50820 · microsoft/TypeScript
This implements support for the Stage 3 Decorators proposal targeting ESNext through ES5 (except where it depends on functionality not available in a specific target, such as WeakMaps for down-level private names). The following items are not currently supported: --emitDecoratorMetadata, as metadata is currently under discussion in https://github.com/tc39/proposal-decorator-metadata and has not ye
Feature/add documents by halsk · Pull Request #24 · cocoa-mhlw/cocoa
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Incremental static regeneration by kirkness · Pull Request #1028 · serverless-nextjs/serverless-next.js
Adds Incremental Static Regeneration! Related issues #804 #995 Add unit tests for regeneration logic Test assumptions around how rollup.js bundles dynamic imports as per @dphang's comment Add e2e tests for ISR fallbacks Setup deployment of infra via serverless Add e2e tests Make code for ISR possible to be agnostic to the provider as per @dphang's comments Only pass what's needed to the queue as p
Pull Request ごとに Cloud Run を作って Identity-Aware Proxy を使ってセキュアに公開する方法
こんにちは。MIXI 開発本部 SREグループの riddle です。 以前 Flutter on the Web と WidgetBook をGCSを使って Pull Request 単位にセキュアに公開する | MIXI DEVELOPERS という記事を書きましたが、今度は Cloud Run で似たことをやってみます。 単に Cloud Run を Pull Request ごとに作るのは簡単ですが、Identity-Aware Proxy を使うところが難しいのでそこに焦点をあてて紹介します。 ＜目次＞ 作った全体の構成仕組み 2.1 Developer が https://XXXXX.example.com にアクセス 2.2 Identity-Aware Proxy が Developer の認証を行う 2.3 Nginx による Cloud Run へのリバースプロキシ 2
esm: Unflag --experimental-modules by guybedford · Pull Request #29866 · nodejs/node
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
なぜGitHubではPull requestと言うのでしょうか。"Push" requestの方がしっくり感じられませんか？
回答 (3件中の1件目) Pull request はもともと Linux カーネル開発で言われていた用語です。そもそも git 自体、Linux カーネルのために作られたツールなので、さまざまな用語がカーネル開発のワークフローに由来しています。 カーネル開発で自分がなにかの機能を開発するために独自のブランチにコミットしてたとします。これをほかのレポジトリ……たとえば Linus のもつメインのレポジトリにとりいれてほしいとき、このブランチにあるものを取り込んでください、とお願いするのが pull request です。これに対してリクエストをうけいれるときには、じっさいに [code...
Add WASI based WebAssembly support by kateinoigakukun · Pull Request #5407 · ruby/ruby
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Pull Request をまとめてくれる Grouped version updates for Dependabot が公開ベータとなりました | DevelopersIO
Pull Request をまとめてくれる Grouped version updates for Dependabot が公開ベータとなりました こんにちは、CX事業本部 Delivery部の若槻です。 Dependabot version updates を使用すると、リポジトリ内で管理している依存関係のアップデート対応を自動化することができます。 先月のアップデートで、version updates によりオープンされる Pull Request をグループ化する機能（Grouped version updates）が公開ベータで利用可能となりました。 これにより、今までは更新対象のパッケージが多い場合は Pull Request が大量にオープンされたり、依存関係の解決でコンフリクトが発生する場合がありましたが、Grouped version updates によりこれらの問題を解
はじめにSHIFTからシステム・アイに出向中の斎藤です。少し前にECSでECRからdocker pullをしようとした際にエラーが出てかなり解決に時間がかかってしまいましたので、その時の状況と解決方法を共有します。 エラー内容ECSでECRからdocker pullをしようとすると、以下のようなエラー文が出ました。 ResourceInitializationError: unable to pull secrets or registry auth: execution resource retrieval failed: unable to retrieve ecr registry auth: service call has been retried 3 time(s): RequestError: send request failed caused by: Post https
IntelliJ を使って Pull Request をレビューする - ドキュメントを見たほうが早い
言いたいことはタイトルが全て。 IntelliJ には GitHub と連携して Issue や Pull Request を見れる機能があり、しっかりコードレビューする際は GitHub のサイト上ではなく IntelliJ 上で行うようにしている。この IntelliJ でレビューを行うことはサイト上で行うよりも体験が良く、便利に思っていた。使い始めた頃は Pull Request の一覧やコメントを書く程度だったが、先日のバージョン2020.2のアップデートでついにレビュー関連の機能が完全にサポートされたので、これを機に IntelliJ 上でレビューを行う体験の何が良いかについて紹介したい。 blog.jetbrains.com 便利だと感じている点は4つ。以下説明していく。 手元に Checkout せずに済むこと 定義/参照ジャンプなどの IntelliJ の機能が使えること
esm: add import.meta.dirname and import.meta.filename by jsumners · Pull Request #48740 · nodejs/node
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
Add cargo builder for rust extensions by ianks · Pull Request #5175 · rubygems/rubygems
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session. You switched accounts on another tab or window. Reload to refresh your session. Dismiss alert
自作PC 本格水冷ガイド ExtraⅢ ラジエータファン Push？ or Pull？ - みかん食べながら
本格水冷はパーツの組み合わせ・配置を自由にできる反面、どのビルドにするのが良いか迷う点も多くあります。 前回のExtraⅡでは冷えない原因をご紹介しましたが、今回はラジエータファンの取付向きがPush方向とPull方向で、どちらが冷やせるのかご紹介しようと思います。 Push方向 Pull方向 Push VS Pull 検証 検証環境 OC設定 レギュレーション 検証結果 Push方向 ラジエータに対して風を吹き付ける方向に取り付けるのがPush方向。 多くの人がPush方向で取り付けてるのではないでしょうか。 LEDで光るファンは吸気面でないとイルミネーションが見えないモデルもありますし、排気面はケーブルや軸を固定する4本のシャーシが見えてしまうので、見栄えからの理由が強いように思います。 稀にPushのが冷えるという意見も見かけるので、冷却力の観点からも選ばれてるのかもしれません。 P
CircleCI は push と pull request 両方をトリガーにすることができない CircleCI は、デフォルトでは push をトリガーにしてビルドするようになっている。 push がトリガーのほうが良いこともあるのだが、GitHub であれば pull request の作成や更新をトリガーにしてほしいこともある。 pull request でのみビルドしたい場合は、 Only build pull requests というオプションを有効にすればいい。 ただ、これはデフォルトブランチへの pull request とデフォルトブランチへの push、タグがビルド対象になるようで、これだけでは不足することもある。 push、pull request の両方を対象にビルドできるようにしたい。 そんな場合は、GitHub Actions で pull request のイ
j次のブックマーク
k前のブックマーク
lあとで読む
eコメント一覧を開く
oページを開く
今すぐできるレビュワーに優しいPull Requestをつくる7つのポイント - every Tech Blog
Rancher Desktopを入れたらdocker pull出来なかった話
node/pull-requests.md at master · nodejs/node · GitHub
Gitのおすすめ設定。pull.ff onlyでpullを便利に | tech-broccoli.life
H&M is forced to pull 'sexualised' school uniform advert
GAS から Github に Pull Request を出してみた - LiBz Tech Blog
Cloud Run で Pull Request 毎の確認環境を作ろう
ECSでECRからdocker pullをする際に気を付けること｜SHIFT Group 技術ブログ
CircleCI 2.1 の pipeline を push と pull request で実行する