ruby-gnome2-devel-ja

[ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて

[S. K. <ch...@ll...>]

こんにちは、北川（kitachro）です。

件名の通り、今公式リポジトリの方にプルリクエストしているコミットについて、
英語でコメントをやりとりするのがちょっとつらいので、こちらに書かせてもらい
ます。また、あとであちらにもまとめの返答をするつもりです。

まず、set_size_requestメソッドを付け加えたことについてですが、自分のWindows７
の環境では、変更前のスクリプトを実行すると、TextViewが最低限の縦幅（60〜70
ピクセル？）でしか表示されなかったので、大きめに表示した方が動作がわかりやす
いかなと思って追加しました。どうしても追加したいというものではないので、ない
方がいいということであれば消しますがどうでしょう？

それからrebaseについてですが、こちらのmasterブランチで
git rebase upstream/master
を実行するということでいいでしょうか？

gitでの共同作業の作法がまだきちんとわかってなくて申し訳ないですが、よろしくお
願いします。

Re: [ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて

[Kouhei S. <ko...@co...>]

須藤です。

In <201...@ll...>
  "[ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて" on Thu, 28 Aug 2014 07:02:44 +0900,
  "S. Kitagawa" <ch...@ll...> wrote:

> 件名の通り、今公式リポジトリの方にプルリクエストしているコミットについて、
> 英語でコメントをやりとりするのがちょっとつらいので、こちらに書かせてもらい
> ます。また、あとであちらにもまとめの返答をするつもりです。

はい！それで大丈夫です！

> まず、set_size_requestメソッドを付け加えたことについてですが、自分のWindows７
> の環境では、変更前のスクリプトを実行すると、TextViewが最低限の縦幅（60〜70
> ピクセル？）でしか表示されなかったので、大きめに表示した方が動作がわかりやす
> いかなと思って追加しました。どうしても追加したいというものではないので、ない
> 方がいいということであれば消しますがどうでしょう？

なにか理由があれば追加してもよいと思っています。
で、わかりやすくなる、というのは追加するのに妥当な理由なので
アリだと思っています。なので、今回のやつは追加した状態のもの
をとり込もうと思います。


その上での話なんですけど。。。
面倒でなければ、「GTK+ 3対応です！」というコミットとはわけて
もらえると助かります。GTK+ 3に対応しようとするとこの変更も必
要なの！？関係なさそうだけど。。。と思ってしまうから、という
のが理由です。

さらに面倒でなければ、という話なんですが、コミットだけでなく
pull requestもわけてもらえると確認する方としては助かります。

次の場合よりも、

  1. GTK+ 3に対応するpull request
     （GTK+ 3対応の変更とわかりやすさの改善の両方を含む）

この場合の方が確認しやすいということを言いたいのですが、うま
く伝わるように説明できている気がしませんね。。。

  1. GTK+ 3に対応するpull request
     （GTK+ 3対応の変更を含む）
  2. わかりやすさを改善するpull request
     （わかりやすさの改善を含む）



まぁ、いろいろ書いたのですが、今のRuby-GNOME2プロジェクトは
確認する側が便利になるよりも、pull requestを送る側の面倒が少
なくなる方を優先したいと思っているので、「あ、これは面倒そう
だなぁ」と思ったら無理して↑に書いている方針でやらなくても大
丈夫です！

> それからrebaseについてですが、こちらのmasterブランチで
> git rebase upstream/master
> を実行するということでいいでしょうか？

あ、少し違います。そうか、北川さんのリポジトリーでもブランチ
を切っていますもんねぇ。

すでに

  % git remote add upstream gi...@gi...:ruby-gnome2/ruby-gnome2.git

を実行している前提で、次のコマンドを実行するとよいです。

  % git checkout master
  % git rebase upstream/master
  % git checkout bindingsrb
  % git rebase master

最初のrebaseで北川さんのリポジトリーのmasterを最新にして、
次のrebaseで北川さんのリポジトリーのmasterをベースに
bindingsrbブランチの変更内容を適用します。

rebaseしているのでpushするときは--forceを付けます。

  % git checkout bindingsrb
  % git push --force

git mergeじゃなくてgit rebaseする理由なんですが、pull
requestを送る用のブランチ（トピックブランチ）にとってマージコ
ミット(*)はトピック外のコミットだからです。トピック外のコミッ
トがあると確認する側が「ん？」と思いやすくなります。

(*) 例: https://github.com/kitachro/ruby-gnome2/commit/e6d187f9f8edf3d95a8a8f687090c9409651ad6b


と、これもいろいろ書いたのですが、面倒そうならそう言ってもら
えればやらなくてもよいです！今の状態で取り込みます！
理由はpull requestを送る側の面倒が減る方を優先したいからです。

> gitでの共同作業の作法がまだきちんとわかってなくて申し訳ないですが、よろしくお
> 願いします。

いえいえ！Gitはそんなに簡単ではないのでしょうがないです！

Re: [ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて

[S. K. <ch...@ll...>]

北川です。

返信ありがとうございます。せっかく詳しく教えていただいたので、
プルリクエストをやり直す方向で行きますね。

rebaseでできるだけ本家に追随した状態で目的別にプルリクエストを
出すという形でいいのかなと理解してます。

そこで一度プルリクエストを引っ込めたいのですが、その場合Closeの
実行でいいのでしょうか？

Re: [ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて

[S. K. <ch...@ll...>]

北川です。

コミットとプルリクエストをやり直すために#245のリクエストは閉じさせて
もらいました。一つ前のメールの質問はスルーしていただけるとありがたいです。

よろしくお願いします。

"S. Kitagawa" <ch...@ll...> wrote:
(2014/08/30 15:41)

>北川です。
>
>返信ありがとうございます。せっかく詳しく教えていただいたので、
>プルリクエストをやり直す方向で行きますね。
>
>rebaseでできるだけ本家に追随した状態で目的別にプルリクエストを
>出すという形でいいのかなと理解してます。
>
>そこで一度プルリクエストを引っ込めたいのですが、その場合Closeの
>実行でいいのでしょうか？

Re: [ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて

[Kouhei S. <ko...@co...>]

須藤です。

In <201...@ll...>
  "Re: [ruby-gnome2-devel-ja] 現在プルリクエストしているコミットについて" on Sun, 31 Aug 2014 17:31:53 +0900,
  "S. Kitagawa" <ch...@ll...> wrote:

> コミットとプルリクエストをやり直すために#245のリクエストは閉じさせて
> もらいました。一つ前のメールの質問はスルーしていただけるとありがたいです。

やり直し、ありがとうございました！
こちらからのリクエストを反映してくれていたので取り込みました！

面倒だったとは思いますが、確認しやすくなっていました。
ありがとうございます！