並び順

ブックマーク数

期間指定

  • から
  • まで

121 - 160 件 / 224件

新着順 人気順

"Pull Request"の検索結果121 - 160 件 / 224件

  • 解読に時間がかかるPull Request - りまりまだんの本拠地

    アプリケーションの保守をやっていると、色んなPull Requestを見る機会がある。レビューする/しないは関係ない。自分の変更を入れる前に過去の実装経緯を確認し、デグレリスクがないか・暗黙的なコンテキストがないか確認したいのだ。もちろんレビュアーをやることもある。解読に時間がかかるPull Requestの傾向が見えてきたのでメモする。 変更点が多い 散々言われているので割愛。コードフォーマッタを途中で導入する場合、コードフォーマット用のPull Requestを作成してほしい。 説明書きやコメントが何もない 保守する側からすると最悪。Issueも真っ白だとヘイトが増してしまう。「今だけ乗り切れば良い」とか、「自分が分かってれば良い」という考えなのかもしれない。ただ、いつかは担当は変わる。そしてコードはメンテされる時間のほうが長い。 せめて何故この変更が必要なのかに関する説明書きは欲しい

      解読に時間がかかるPull Request - りまりまだんの本拠地
    • feat: Import cargo-add into cargo by epage · Pull Request #10472 · rust-lang/cargo

      Motivation The reasons I'm aware of are: Large interest, see Add cargo-add (from cargo-edit) to cargo proper #5586 Make it easier to add a dependency when you don't care about the version (instead of having to find it or just using the major version if thats all you remember) Provide a guided experience, including Catch or prevent errors earlier in the process Bring the Manifest format documentati

        feat: Import cargo-add into cargo by epage · Pull Request #10472 · rust-lang/cargo
      • Incorporating feedback in your pull request - GitHub Docs

        Applying suggested changes Other people can suggest specific changes to your pull request. You can apply these suggested changes directly in a pull request if you have write access to the repository. If the pull request was created from a fork and the author allowed edits from maintainers, you can also apply suggested changes if you have write access to the upstream repository. For more informatio

          Incorporating feedback in your pull request - GitHub Docs
        • GitHub の Pull Request はマージ方法が 3 パターンあるって知ってました?

          自主的20%るぅる > 最新記事 > 技術ネタ > GitHub の Pull Request はマージ方法が 3 パターンあるって知ってました? こんにちは、江嵜です。 GitHub の Pull Request つかってますか? みなさん、 GitHub 使ってますか? 先日プライベートレポジトリの利用が無料で出来るようになりましたね! これで、これまで 「プライベートレポジトリを無料で使いたいから GitLab (または BitBucket) にするかー」 みたいな感じだった方々も GitHub を使いやすくなりましたね! さて、GitHub といえば Pull Request ですね。 他のサービスでも Merge Request 等の名前で同じような機能が提供されていますが、 要は git で平行して開発していたものを一つにまとめる作業 (これがマージ) の管理を カンタンに実行

            GitHub の Pull Request はマージ方法が 3 パターンあるって知ってました?
          • 【Github actions】DependabotのPull RequestでSecretsが参照できずワークフローがFailになった場合の対処 - Simple minds think alike

            Dependabot のPull Request(以下PR)が作られた際に開始したGithub Actionsワークフローが Secrets を参照できずに失敗していたので原因を調べてみました。 2021/3/1から適用になった以下のUpdateが影響していて、 Dependabot から実行される Github Actionsワークフローは読み取りだけが可能な GITHUB_TOKEN のみ使うことができ、いかなる Secrets も使えなくなるという変更が原因でした。 github.blog なので、例えばpushイベントトリガーで実行されるワークフローの中で Secrets として追加しておいたPersonal access tokensを使って、取得したカバレッジのサマリをコメントで追加したり、自動でラベルを追加するといった書き込み(write)権限が必要な場合は、ワークフローが落

            • Pull request auto-merge public beta

              December 16, 2020 Pull request auto-merge is now rolling out as a public beta! With auto-merge, pull requests can be automatically merged when all requirements for merging are met. No more waiting for long checks to finish just so you can press the merge button! To use auto-merge, a repository maintainer or admin must first toggle on the repository setting to allow auto-merge (see steps). Then any

                Pull request auto-merge public beta
              • Bash installer by itchyny · Pull Request #256 · Homebrew/install

                Hi, I looked into the installer and translated into bash one line by one. Needs some more works (not sure how to compare version string.. maybe I'll implement this with arrays parameter substitution works). I love Homebrew, thank you all. Why? resolves Port installer to Bash #217 Future versions of macOS won’t include scripting language runtimes by default, and might require you to install additio

                  Bash installer by itchyny · Pull Request #256 · Homebrew/install
                • RFC: The Next.js "App Router", React Server Component & "SSR with Suspense" story by phryneas · Pull Request #9 · apollographql/apollo-client-nextjs

                  Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community. Pick a username Email Address Password Sign up for GitHub By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails. Already on GitHub? Sign in to your account

                    RFC: The Next.js "App Router", React Server Component & "SSR with Suspense" story by phryneas · Pull Request #9 · apollographql/apollo-client-nextjs
                  • src,doc: Experimental support for SEA by mhdawson · Pull Request #42334 · 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

                      src,doc: Experimental support for SEA by mhdawson · Pull Request #42334 · nodejs/node
                    • 【GitHub】Pull Requestの手順 - Qiita

                      Git勉強会(社内勉強会)でできなかった、Pull Requestについての説明です。 Pull Request の手順 単純にソースコードを取得したいならgit clone http://{対象のGitHubリポジトリ}で大丈夫です。もしコードの内容を修正したい場合はリポジトリをForkする必要があります。 対象のGitHubリポジトリをFork ローカルへclone(クローン) ローカルリポジトリで新規ブランチを作成 修正を加える(コミット) 作成したブランチをpush(プッシュ)する Pull RequestをGitHub上で作成 1. 対象のGitHubリポジトリをFork 対象のGitHubリポジトリをブラウザ上でアクセスします。 んで、そこにあるForkってボタンをクリック。 自分のアカウントに対象のリポジトリがForkされます。 Forkって何? 基本的にはgit clone

                        【GitHub】Pull Requestの手順 - Qiita
                      • GitHubの scheduled reminders を設定して Pull Requestのレビュワー指定/ 再レビュー依頼/ Conflict等々をSlack通知する - Qiita

                        GitHubの scheduled reminders を設定して Pull Requestのレビュワー指定/ 再レビュー依頼/ Conflict等々をSlack通知するGitHubSlack みなさん、Scheduled Reminders 使ってますか? Scheduled Reminderとは, GitHubが Pull Pandaを買収して実現したGitHubのPR通知機能です。 GitHub acquires Pull Panda—a better way to collaborate on code reviews レビュワーのアサイン通知、approveやrequest change、conflictの発生などをDMで受け取れます。 (その点で、リポジトリを subscribeしてチャンネルに投稿するGitHubのSlack Botとは異なります) 手順 (私の環境では既にイ

                          GitHubの scheduled reminders を設定して Pull Requestのレビュワー指定/ 再レビュー依頼/ Conflict等々をSlack通知する - Qiita
                        • Add signed ids to Active Record by dhh · Pull Request #39313 · rails/rails

                          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 signed ids to Active Record by dhh · Pull Request #39313 · rails/rails
                          • Trigger workflow only on pull request MERGE · community · Discussion #26724

                            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

                              Trigger workflow only on pull request MERGE · community · Discussion #26724
                            • Pull Request何個で出来るかで計画を立てる - hitode909の日記

                              これいつまでにできますか?と考えるとき、1日1個ずつPull Requestを出してマージしていけるとして、何個のPull Requestを出せば完成するか?と考えることが多い。 進め方の前提 Pull Requestでの変更は、ひとことで言い表せる単位の粒度にしたい ❌ 〇〇を✗✗にして、△△は□□に、一方☆☆は**にする ⭕ 〇〇を✗✗にする レビューが通る前に次々と実装していくと手戻りが大きくなる可能性があるので、並行して手を付けずに、1個ずつ順番にやりたい 大きいPull Requestを作ってしまうと理解が困難になり、レビュー漏れの恐れもあるので、無理なく見れる範囲にしたい リソースの前提 実装する時間と、レビューから返ってきたときに返事したり、必要ならコードを修正する時間は確保したい 丸一日すきまなくミーティングだとソフトウェア開発する時間がなくなるため レビュワーは固定で1人

                                Pull Request何個で出来るかで計画を立てる - hitode909の日記
                              • src: add initial support for single executable applications by RaisinTen · Pull Request #45038 · 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

                                  src: add initial support for single executable applications by RaisinTen · Pull Request #45038 · nodejs/node
                                • Renovateでつくられるpull requestのgithub .comへのリンクはドメインがtogithub .comに置き換えられている - hogashi.*

                                  Mend Renovate: Automated Dependency Updates を使うと、リポジトリに勝手に pull request を作ってくれるけど、その description に書かれる github.com 以下のリンクは、ドメインが ( github.com ではなく) togithub.com になっている。 これはバックリンクが作られてしまうことを防ぐためのものらしい。確かに、 GitHub 内で他の issue などのリンクを貼ると貼られた側にバックリンクが誕生して便利だけど、 Renovate のような機械的に大量に p-r を作るタイプのものだと困りそう。 github.com ちなみに togithub.com は誰が運用しているのか、と思って whois したら Redacted for Privacy になっていた。が、 Renovate のコードには

                                    Renovateでつくられるpull requestのgithub .comへのリンクはドメインがtogithub .comに置き換えられている - hogashi.*
                                  • [Experiment] Context Selectors by acdlite · Pull Request #20646 · facebook/react

                                    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

                                      [Experiment] Context Selectors by acdlite · Pull Request #20646 · facebook/react
                                    • WIPの代わりにDraft Pull Requestを利用する (GitHub) - Qiita

                                      Draft Pull Requestをリリースしました を読めばおわりです Draft Pull Requestとは GitHub Flow などを利用してチーム開発をしていると、すぐマージはできなくても Pull Request(PR) を出したくなる場面があると思います。 そのような時は、 WIP(Work in Progress) を利用することが多いでしょう。 作業中の PR のタイトルに WIP: PR title や [WIP] RP title のようにつけるのが一般的です。 WIP を利用することで以下のような恩恵があります 作業ログやコメントを残せるため、思考の整理になる PR を見ることでコンテキストをレビュワーと共有できるため、レビューが楽になる また、マージ目的ではなく他のメンバーからのフィードバックをもらうために PR を出すこともあります。 このような場合に便利

                                        WIPの代わりにDraft Pull Requestを利用する (GitHub) - Qiita
                                      • Add delegated type to Active Record by dhh · Pull Request #39341 · rails/rails

                                        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 delegated type to Active Record by dhh · Pull Request #39341 · rails/rails
                                        • GitHub - AlexSim93/pull-request-analytics-action: Generates detailed PR analytics reports within GitHub, focusing on review efficiency and team performance.

                                          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 - AlexSim93/pull-request-analytics-action: Generates detailed PR analytics reports within GitHub, focusing on review efficiency and team performance.
                                          • GitHub - Codium-ai/pr-agent: 🚀CodiumAI PR-Agent: An AI-Powered 🤖 Tool for Automated Pull Request Analysis, Feedback, Suggestions and More! 💻🔍

                                            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 - Codium-ai/pr-agent: 🚀CodiumAI PR-Agent: An AI-Powered 🤖 Tool for Automated Pull Request Analysis, Feedback, Suggestions and More! 💻🔍
                                            • 11章のコードをActix-Web 2.0.0に移植する by KeenS · Pull Request #2 · ghmagazine/rustbook

                                              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

                                                11章のコードをActix-Web 2.0.0に移植する by KeenS · Pull Request #2 · ghmagazine/rustbook
                                              • Replace Webpacker with importmapped Hotwire as default JavaScript setup by dhh · Pull Request #42999 · rails/rails

                                                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

                                                  Replace Webpacker with importmapped Hotwire as default JavaScript setup by dhh · Pull Request #42999 · rails/rails
                                                • Create Pull Request - GitHub Marketplace

                                                  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

                                                    Create Pull Request - GitHub Marketplace
                                                  • Preview environments per Pull Request using AWS CDK and Github Actions

                                                    Preview environments per Pull Request using AWS CDK and Github Actions TL;DR The code is 👉 here 👈. The rest of this post gives information and context about it. 📖 Table of contents 👋 Introduction 🎯 Target ✅ Prerequisites ☁️ AWS Architecture CDK File structure cdk.json bin/app.ts lib/awesome-stack.ts CLI yarn bootstrap yarn deploy yarn destroy 🤖 Github automation Github Deployments API Github

                                                      Preview environments per Pull Request using AWS CDK and Github Actions
                                                    • [GHSA-27h2-hvpr-p74q] Request to reject CVE: jsonwebtoken has insecure input validation in jwt.verify function by MichaelErmer · Pull Request #1595 · github/advisory-database

                                                      Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community. Pick a username Email Address Password Sign up for GitHub By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails. Already on GitHub? Sign in to your account

                                                        [GHSA-27h2-hvpr-p74q] Request to reject CVE: jsonwebtoken has insecure input validation in jwt.verify function by MichaelErmer · Pull Request #1595 · github/advisory-database
                                                      • Pull request review (prr)

                                                        Last month I developed a new tool that brings mailing-list style code reviews to github. The tool was born out of frustration with github’s point and click web UI. Since I do a lot of code review, the tabbing between my text editor and my browser became a constant source of friction. Ideally, I wanted to stay in my text editor so that my code writing and code reviewing workflows could be unified.

                                                        • [GitHub] Pull Requestでファイルに直接コメントできるようになりました(公開ベータ版) | DevelopersIO

                                                          こんにちは、CX事業本部 Delivery部の若槻です。 このたびののGitHubアップデートにより、プルリクエストでファイルに直接コメントできる機能(公開ベータ版)が導入されました。 このアップデートにより、従来の行ベースのコメントに加えて、ファイル全体にコメントできるようになりました。これにより特に次のようなファイル差分に対するコメント追加が便利になります。 名前が変更されたファイル バイナリファイル(画像など) 削除されたファイル 試してみた プルリクエストの[Files changed]タブで、ファイルごとに[Comment on this file]ボタンが表示されます。 ボタンをクリックすると、コメント欄が表示されます。コメントを記入して保存します。 コメントが追加されました。 [Conversation]タブでは、次のように表示されます。 従来のコメントと同様に、コメントの返

                                                            [GitHub] Pull Requestでファイルに直接コメントできるようになりました(公開ベータ版) | DevelopersIO
                                                          • Remove abuse enabling language by jacobherrington · Pull Request #2690 · ruby/www.ruby-lang.org

                                                            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

                                                              Remove abuse enabling language by jacobherrington · Pull Request #2690 · ruby/www.ruby-lang.org
                                                            • Use Prettier 3.0 by default by sosukesuzuki · Pull Request #3069 · prettier/prettier-vscode

                                                              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

                                                                Use Prettier 3.0 by default by sosukesuzuki · Pull Request #3069 · prettier/prettier-vscode
                                                              • Open a GitHub Pull Request From Your Terminal

                                                                You finally did it! You finished writing your new feature and it's fully tested. You write up a snazzy commit message and hit enter. Your command line is telling you that you have a clean branch and you want to share it with the world RIGHT NOW! Unfortunately, you have a few mind-numbing steps to take before everyone can bask in the glory of your code. First, you have to push the branch to your re

                                                                  Open a GitHub Pull Request From Your Terminal
                                                                • Pull Request Analytics - GitHub Marketplace

                                                                  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 Analytics - GitHub Marketplace
                                                                  • TS to JSDoc Conversion by tcc-sejohnson · Pull Request #8569 · sveltejs/svelte

                                                                    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

                                                                      TS to JSDoc Conversion by tcc-sejohnson · Pull Request #8569 · sveltejs/svelte
                                                                    • Pull Request Retargeting

                                                                      May 19, 2020 Previously, when a pull request's head branch was merged and deleted, all other open pull requests in the same repository with a base branch of the merged branch would be automatically closed. Now, instead of automatically closing these open pull requests, the pull requests will be retargeted—the base branch of each pull request will be updated to the merged pull request's base branch

                                                                        Pull Request Retargeting
                                                                      • GitHub – Pull Requestをレビュー【コメントの書き方と修正依頼】

                                                                        本番、または共有のブランチに担当者のコミット分をマージする場合には、メンバーのレビューを通すことが理想です。 この記事では、GitHubでPull Requestをレビューする方法を解説します。GitHubでは、コード1行ごとにコメントを入れることができるなど、レビューを行う負担が最小限になるよう工夫されています。 Pull Requestをレビューする方法GitHubでPull Requestをレビューするには以下の手順で行います。 『GitHub.com』にアクセスし、対象のリポジトリから「Pull request」を選択します。 一覧から、確認したいPull requestを選択します。 あなたがレビュアーとしてアサインされている場合、緑のボタン「Add your review」が表示されます。クリックしてレビューを開始します。 コミット内容についてコメントを書きます。GitHubで

                                                                        • Make npm install scripts opt-in by tolmasky · Pull Request #488 · npm/rfcs

                                                                          Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community. Pick a username Email Address Password Sign up for GitHub By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails. Already on GitHub? Sign in to your account

                                                                            Make npm install scripts opt-in by tolmasky · Pull Request #488 · npm/rfcs
                                                                          • Pull Request を作るのに一時間以上かかったら捨ててる - Object.create(null)

                                                                            一時間は大体の目安でちゃんと測ってない. PR の作成に時間がかかるときは, 何らか良くないことが起きている可能性が高い 試行錯誤を繰り返している 変更の規模が過大になっている 良くないことが起きているなら, そのまま続けて余計なコストをかけるよりも捨てた方が良い 試行錯誤を繰り返していたなら, そこまでは勉強か練習だと思って捨てる 変更の規模が過大になっていたなら, 分割して作り直すために捨てる 捨てることを躊躇わない 時間をかけるほど引き返しづらくなるので, 一時間くらいで打ち切る 一時間の手戻りなら一時間で取り返せる. なんなら知識が増えている分もっと短くて済む 思ったより進んでいなくても, そのまま思っていたところまで進めようとはせずに諦めて捨てる. とにかく良くないことが起きている可能性が高いので立ち止まるべきで, 思ったところまで進められる (良くない状態に陥らなくなる) 状態

                                                                              Pull Request を作るのに一時間以上かかったら捨ててる - Object.create(null)
                                                                            • GitHubでPull Requestの宛先に誤って40万人のチームを指定してしまい祭り状態に | スラド IT

                                                                              タレこみ子の元にも先週末に大量のメールが来たのだが、EpicGamesのGitHubリポジトリに出されたPull Requestで、レビューの依頼先として誤って「@EpicGames/artv2-admin」「@EpicGames/developers」「@EpicTeamAdmin」というEpicGamesの全開発者アカウントを含むチームが指定された事件があり、約40万人に大量の通知が届く騒動があった(GitHub、GIGAZINE)。 意味不明の通知を受け取ったユーザーは、Pull Requestに「これはなんだ?」「通知を止めるにはどうすればいい?」といったコメントを残し、コメントがさらなる通知を飛ばし、悪乗りしてコメントする人も現れ…という悪循環で、最終的にスパムとしてPull Requestがクローズされるまでに100件以上のメールが飛ぶ事態となってしまったようだ。 タレこみ子は

                                                                              • Pull Requestに関する意見 - mactkg-pub

                                                                                レビュイーの方がレビュワーよりもPull Requestにまつわるコンテキストを多く持っているため、レビュイーがレビュワーに向けて過不足なく説明をすべき。

                                                                                  Pull Requestに関する意見 - mactkg-pub
                                                                                • Better words by richsalz · Pull Request #12089 · openssl/openssl

                                                                                  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

                                                                                    Better words by richsalz · Pull Request #12089 · openssl/openssl