タグ

pull_requestに関するopparaのブックマーク (4)

  • ふだんの開発で小さなPRに分けるために考えていること - 私が歌川です

    ふだんの開発でPRを出すときに考えていること - 私が歌川です の続編です。小さなPRを出す、という話がありましたが、どうやって小さくしているのか、というのを書きます。 どういうメソッドが欲しいか考える Aという情報をBという画面に出したい Aを取得するメソッドがないので、それを作る必要がある (メソッドCとする) というときに、メソッドCを作るPRをまず先に出して、マージされたら続いてCを使って得られる情報Aを画面Bに出すPRを出す、というのをやります。 実装方針を考えて既存実装を眺めてからそういう作戦を立てることもあれば、実装しているうちにメソッドCが足りないな、と気づいて別ブランチでPRを作ってそちらを先に出す、ということもあります。 このとき、1つのPRで両方やろうとすると、メソッドCの実装のレビューと、情報Aを画面Bに出す実装のレビューが同じPRで行われることになります。メソッド

    ふだんの開発で小さなPRに分けるために考えていること - 私が歌川です
  • 「Pull Request」 はオープンソースに限らず使える優れた開発フローだ - 肉とビールとパンケーキ by @sotarok

    チーム開発において、「チケット/Issue」「TDD」「コードレビュー」など、ソースコードの変更に対する効果的な開発フローについてよく考えるのだけど、なんにしてもこのあたりは非常に課題が多く、各社各コミュニティで色々なやり方が模索されているポイントだと思う。 で、まぁご多分に漏れず僕もよく考えるわけだけど、現状その過程で Pull Request こそが非常に効果的なのではないか、と思うので、ちょっとまとめてみようかと思う。 もちろん、言うまでもないようなことだよ、という人もいるかもしれないけど、そういう人がたくさんいると、非常に喜ばしいことだね。 Pull Request とは GitHub でこう呼ばれているので、こう呼ぶことにするが、ここでは、複数のリポジトリ/ブランチ間でのオープンな patch のやりとりのことだと考える。 あと、自分が使っているのが Git なので、ここでは G

    「Pull Request」 はオープンソースに限らず使える優れた開発フローだ - 肉とビールとパンケーキ by @sotarok
  • GitHub直伝 プルリクエスト活用の3つのコツ

    GitHubの特に重要な機能である「プルリクエスト」の活用方法についてGitHub社内でのノウハウが公式ブログの記事になっていました。GitHubが今回更新をしたAboutページの開発でも2ヶ月の間に10人のメンバーが130のコミットと91のコメントのやりとりがブランチ上で行われていました。 GitHubberによる講演などでもプリリクエストが重要な機能であると強調されているようです。 記事によるとプルリクエストは新しいアイデアについてのディスカッションを生み、協力してくれる人を見つける為のとても良い方法との事で活用するコツとして以下の3つの点を紹介しています。 プルリクエストはなるべく早く起こす プルリクエストは機能についての意見交換をする良いきっかけになります。コードの修正が終わっていなくてもなるべく早くプルリクエストをする事で、最後にまとめてフィードバックをするのではなく発展的にコメ

    GitHub直伝 プルリクエスト活用の3つのコツ
  • 実践!Pull Request対応 - Meltdown Countdown rev.

    Ginatra というものがあります。SinatraとGritでできたGitリポジトリビューアです。Githubクローンというよりは、進化版gitwebといった感じ。 機能は良いのですが、いかんせん見た目が今一歩。これをforkして、Twitter Bootstrapを使って簡単によさげな見た目にしていたわけです。 marutanm/ginatra · GitHub 見た目のほかには、erbだったのをhaml化したりとか、ちょっとルーティング変えたりとか。 そうこうしていたらPull Requestがきたのです。なぜか家のほうではなく、こっちに。なんとも喜ばしいですね!! せっかくなので、Pull Requestがきたときにどう対応するか、まとめてみましょう。 Auto merging 一番かんたんなのは、 Merge Buttonを使うことですね。 画像の右下にあるのが、それ。これを押

    実践!Pull Request対応 - Meltdown Countdown rev.
  • 1