-
Notifications
You must be signed in to change notification settings - Fork 24
to merge branch
Yuka Kato edited this page Jun 22, 2015
·
3 revisions
プルリクエスト(Pull Request、以下 PR と書きます) を送った後には、レビューコメントを受けながらブラッシュアップしていってクオリティを上げ、マージすることになります。
マージという操作は PR の送信者やリポジトリ管理者だけでなく誰でもできるものなのですが、よちよち.rb ではお互いのコード(文章)をよりよいものにして残したいという想いから、「Looks Good To Me (LGTM)」コメントを 2人 以上のメンバーにもらってからマージする というルールで運用しています。
これによって Markdown の記述ミスを指摘してもらえたり、メンバーどうしのコミュニケーションがはかどったりといいことづくめなので、ぜひご協力をお願いします。
- Doorkeeper でお願いしていることがやってあること
- レンダリングされたファイルが崩れていないこと(Markdown を使っていると特に table や list 系は期待通りになっていないことが多いためです)
- typo や表記ミスがないこと(隅から隅まで表記間違いをくまなく探せ!とかいうつもりではなく、間違いだと気づいたところをあえて見過ごしちゃう必要もないよね、といった感覚です)
上記を満たしていれば LGTM として問題ありません。レビューの目安やプルリクエスト作成の参考になさってください。
その他にも、レビュー時に Git および GitHub の使用方法としてベターな方法があると思った場合などはコメントしていただいてかまわないと思っています。そのへんは "勉強会" ですので、知識のシェアという意味でもむしろ遠慮なくやっていただいていいと思います。