Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

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

いまさらですが、誰がマージするのかについて #7

Closed
AriYu opened this issue Nov 14, 2016 · 3 comments
Closed

いまさらですが、誰がマージするのかについて #7

AriYu opened this issue Nov 14, 2016 · 3 comments

Comments

@AriYu
Copy link
Member

AriYu commented Nov 14, 2016

今は適当にマージしている感じがありますが、最低でもプルリクエストを出した人以外でApproveがあって、プルリクエストとApproveした人以外がマージしたほうがいいと思いますが、いかがでしょうか。
build報告は誰でもwelcomで。
こうすると開発には最低でも3人が必要ということになってしまいますが。

@forno
Copy link

forno commented Nov 14, 2016

私はPull-requestを出した人がマージするのが良いのではないかと思っています。

まずPull-requestを出し、他の人がApproveし、Pull-requestを出した人がマージし、即座にマージ後の状態をチェックする。
という流れが良いかと思っております。
これでしたら開発に2人、ということになります。

追記:
Pull-request者がマージするのは、マージ後の状態チェック時に万一問題があった時、一番対応しやすいかと思うからです。
また、開発途中でレビュが欲しくなった時などの、たまに未完の状態で出すpull-requestが誤ってマージされるという問題を回避できます。

@MoriKen254
Copy link
Member

なるほど,よく分かりました,
「Pull-requestを出した人がマージする」で,承知しました.

@forno
Copy link

forno commented Nov 14, 2016

ではPull-requestの方針は以下のように。

Pull-requestを出した人がマージする

Issueを閉じます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants