search

sakura-editor / management-forum

管理・運用向けフォーラム（Issues をフォーラム代わりに使う）
2 stars 0 forks source link

誰がマージすべきか? #49

Closed m-tmatma closed 5 years ago

m-tmatma commented 5 years ago

誰がマージすべきか? https://github.com/sakura-editor/sakura/pull/591#issuecomment-437367189

merge 誰 approve をキーワードに google で検索したところ以下のページが見つかった。 https://github.com/CIR-KIT-Unit03/cirkit_unit03_whole_issue/issues/7

上記では、Pull-requestを出した人で落ち着いたみたいです。 Pull-requestを出した人がメンバーである場合に限定されますが。

KENCHjp commented 5 years ago

上記では、Pull-requestを出した人で落ち着いたみたいです。 Pull-requestを出した人がメンバーである場合に限定されますが。

この認識です。プロジェクトメンバー外の人のRPだとAdminメンバーっすかね。メンバーであればだれでもいい気もします。 各メンバー得手不得手あるやもしれないので。

ds14050 commented 5 years ago

結論ではなく結論に至る観点を見ましょう。

という観点からその結論に至ったようです。適任者を責任者と読み替えれば(※特に担当領域が決められているわけではないサクラエディタプロジェクトではほぼ同じ意味です)、自分の考えと変わりません。

ds14050 commented 5 years ago
berryzplus commented 5 years ago

元も子もない意見かも知れませんがぼくの意見を書いておきます。

結論 ・やりたい人がマージすべきです。

補足 ・やりたい人が複数いる場合は競合が起きます。 その場合の優先順位は1=PRを出した人、2=レビュアー以外のメンテナ、3=レビュアーだと思います。

自分が関わったものは、きっと自分でマージしたいはずだ、という勝手な思い込みに基づくものです。そうじゃない人がいるかも知れません。@ds14050 さんは自分でやりたい派だと思います。ぼくも多分にそういう部分があります。

やりたい人がマージしよう！でいいんじゃないかな、と思います。

ds14050 commented 5 years ago

やりたい人やりたい人では誤解を招くので、berryzplus さんに代わって berryzplus さんの意見をまとめましょう。

  1. 前提。マージする義務を負っている人はいない(強いていえば 3 のレビュア)。
  2. PR を出した人が最優先
  3. 対象分野の面倒を見ている人が２番
  4. 関わった誰かが最後の責任を引き受ける

サクラエディタでは今のところ２番の役目を負っている人はいないと思います(あるいはファイルの新規作成者やコミットが多い人が当てはまるのかもしれません)。

リストに特に異存はありません。

m-tmatma commented 5 years ago

私は誰がマージしてもいいと思います。 PR を出した人でも、レビューした人でも、それ以外の人でもいいと思います。

PR を出した人は、一番詳しいはずですが、 レビューした人は、チェックして ok を 出したので、最低二人が大丈夫と判断したからですし、レビュアー以外でもマージするからには内容を多少なりとも確認しているはずだからです。

誰がマージするかより、approve されてから、実際にマージするまでに多少の時間を置くのが大事だと思います。

レビューに参加してない人が、マージ後にコメントを書くことがたまにあるからです。

ds14050 commented 5 years ago

レビューに参加してない人が、マージ後にコメントを書くことがたまにあるからです。

何度かそういうことをしている自分の事情です。

berryzplus commented 5 years ago
  1. PR を出した人が最優先
  2. 対象分野の面倒を見ている人が２番
  3. 関わった誰かが最後の責任を引き受ける

責任分散の感覚値は 4 : 4 : 2 です。

ds14050 commented 5 years ago

責任分散の感覚値は 4 : 4 : 2 です。

2 の人の存在感の大きさで 1 と 2 の具体的な比率は変わってきそうです。

KENCHjp commented 5 years ago
  1. 前提。マージする義務を負っている人はいない(強いていえば 3 のレビュア)。
  2. PR を出した人が最優先
  3. 対象分野の面倒を見ている人が２番
  4. 関わった誰かが最後の責任を引き受ける

この方向に賛同、まケースバイケースで、「俺がマージしたい」って人もいるかもしれませんし。 最初にPR出した人から、途中変更されて、自分の意図しない方向になってったら、取り込まれるより自分で破棄したいかもしれないし。

m-tmatma commented 5 years ago

PR を出した人がマージするというのが定着してきたので閉じます。