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

やりやすいようにルールを決める・変える・むしろ決めないで曖昧にしておくスレッド #611

Open
1 task
Torokun opened this issue Jan 25, 2014 · 7 comments

Comments

@Torokun
Copy link
Member

Torokun commented Jan 25, 2014

これは「すごい広島」をもっとやりやすくするためにルールを決める・変える・むしろ決めないで曖昧にしておくことを議論するスレッドです。

議題があればコメントください。

いま出ている議題

  • 全てのPull Requestのマージはレビュー2個を必要とするのか、「今日やること宣言」に対するPull Requestだけレビュー2個を必要し、他は1個でよいのか
@eiel
Copy link
Member

eiel commented Jan 25, 2014

「やること宣言に対するレビューをしようと思う人」を増やせるなら、それ意外のものはサクっとマージしたい!

@Torokun
Copy link
Member Author

Torokun commented Jan 26, 2014

そうよねー。運用系(ページ作ったり)のプルリクはサクっとマージしたいよね。2個レビューだと放置状態になる可能性があるからねぇ・・・

@yukilabo
Copy link
Member

そもそも変更内容が理解できなくてレビュー出来ない…

@eiel
Copy link
Member

eiel commented Jan 26, 2014

そもそも変更内容が理解できなくてレビュー出来ない…

それはたしかに…あるので、そういうのはルール外にすべきっすね。

@eiel
Copy link
Member

eiel commented Feb 14, 2014

そういえば 某社はレビューが規定数ついたら自分でマージする。というのがあるそうですね。

ますますマージされなくなる可能性もあるけど。

@Torokun
Copy link
Member Author

Torokun commented Feb 14, 2014

ほー。どうなんだろ、それ。指摘ばっかりで全てを直さないまま、「とりあえずマージ」っていうのも可能になってしまうな・・・。
それでいいんかもしれんけど。

@eiel
Copy link
Member

eiel commented Feb 14, 2014

マージされた後にpull-request だしてもいいし、マージされる前に pull-request をさらにのっけてもいいし。

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