to merge branch

Yuka Kato edited this page Jun 22, 2015 · 3 revisions

プルリクエストをマージするには

プルリクエスト(Pull Request、以下 PR と書きます) を送った後には、レビューコメントを受けながらブラッシュアップしていってクオリティを上げ、マージすることになります。

マージという操作は PR の送信者やリポジトリ管理者だけでなく誰でもできるものなのですが、よちよち.rb ではお互いのコード(文章)をよりよいものにして残したいという想いから、「Looks Good To Me (LGTM)」コメントを 2人 以上のメンバーにもらってからマージする というルールで運用しています。

参考:LGTMとは - はてなキーワード

これによって Markdown の記述ミスを指摘してもらえたり、メンバーどうしのコミュニケーションがはかどったりといいことづくめなので、ぜひご協力をお願いします。

よちよち的 LGTM の条件

  • Doorkeeper でお願いしていることがやってあること
  • レンダリングされたファイルが崩れていないこと(Markdown を使っていると特に table や list 系は期待通りになっていないことが多いためです)
  • typo や表記ミスがないこと(隅から隅まで表記間違いをくまなく探せ!とかいうつもりではなく、間違いだと気づいたところをあえて見過ごしちゃう必要もないよね、といった感覚です)

上記を満たしていれば LGTM として問題ありません。レビューの目安やプルリクエスト作成の参考になさってください。

その他にも、レビュー時に Git および GitHub の使用方法としてベターな方法があると思った場合などはコメントしていただいてかまわないと思っています。そのへんは "勉強会" ですので、知識のシェアという意味でもむしろ遠慮なくやっていただいていいと思います。

lgtms

You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.
Press h to open a hovercard with more details.