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.

Sign up for GitHub

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

Jump to bottom

Pull Requestマージするルール決めたい #15

Open
hanachin opened this issue May 13, 2014 · 7 comments
Open

Pull Requestマージするルール決めたい #15

hanachin opened this issue May 13, 2014 · 7 comments
Labels
bug

Comments

@hanachin
Copy link
Member

すごい広島にのっとって2人以上LGTMでマージでいいかなと思うのですがいかがでしょう
http://great-h.github.io/rule.html

あとgh-pagesブランチに直接pushは避けたいです!
@hanachin hanachin added the bug label May 13, 2014
@hanachin
Copy link
Member Author

2人以上LGTMでマージだと、活発に見る人が3人いないと回らない感ある

@yasulab
Copy link
Member

yasulab commented May 13, 2014

そうね。ルールをそのまま踏襲する必要はなさそう。

@hanachin
Copy link
Member Author

まあ二人で確認出来たほうがよさそうなので(事故を防ぐ的に、あと承認欲求的にも)来週あたりに、watchしてレビューしてくれる人いないか聞いてみます

@yasulab
Copy link
Member

yasulab commented May 13, 2014

了解〜 👌

@hanachin
Copy link
Member Author

レビューしてくれる人が2人集まったらこのルールで、それまでは1人LGTMでも 🆗 にしましょう!

@yasulab
Copy link
Member

yasulab commented May 19, 2014

それまでは1人LGTMでも 🆗 に

良いと思います! 👍

@kimihito
Copy link
Contributor

ルール了解です〜

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@yasulab @hanachin @kimihito