PR template を追加する #15
Conversation
|
あと既存で受けてもらっている人にマージしてもらうかか。 |
|
単に origin master だと自分のリモートの master を見ちゃって、 fork 元は見ない気がする。 https://help.github.com/articles/fork-a-repo/#keep-your-fork-synced |
.github/PULL_REQUEST_TEMPLATE.md
Outdated
| @@ -0,0 +1,27 @@ | |||
| PR を作成する時に Description を書いてみましょう。 | |||
| 各項目に例を載せているので、それに合わせて書いてみて下さい！ | |||
|
|
|||
There was a problem hiding this comment.
Why というか「やりたかったこと」的なセクションがあっても良いのかなと思ったりした。
.github/PULL_REQUEST_TEMPLATE.md
Outdated
| @@ -0,0 +1,27 @@ | |||
| PR を作成する時に Description を書いてみましょう。 | |||
There was a problem hiding this comment.
先の話じゃないけど「Pull Request」って書いておいたほうがいいかも？
もしくは「このプルリクエストでやった作業を分かりやすくするため、Description を書いてみましょう」とか？
.github/PULL_REQUEST_TEMPLATE.md
Outdated
| 動作確認方法 | ||
| --- | ||
|
|
||
| どうやったら、レビュー者がこの PR でやったことを確認できますか？ |
There was a problem hiding this comment.
「このPRでの作業内容の確認方法を書いておくとより親切です。 👍 」とかのが良いかも？
|
指摘してもらったポイントで調整してみた！ |
|
これで出来た。 |
|
内容は良さそうに思いました！ forkした場合に元のブランチのmasterを取ってくる時にはよく |
|
あ、 upstream の方が一般的なのか 🙏 |
|
連絡は一斉通知使うかな？ |
|
そうですね！ |
|
これマージしちゃうね！ |
| 動作確認方法 | ||
| --- | ||
|
|
||
| どうやったらレビュー者が作業内容を確認できるか、確認方をを書いてみましょう。 |
There was a problem hiding this comment.
マージされた PR にコメントするのアレだけど
s/確認方をを/確認方法を/
(あとで時間あいたら PR 出します)
PR template を追加する
PR template を追加する
参加者に最低限レベルで書いてもらうために、 template を追加する。
見出し英語の本文日本語でもいいかなと思ったけど、全部日本語にした。
Why 的なのは書きづらいと思うけど、いるかな？🤔
あと Base ブランチの話しも載せるかちょっと悩んだ
@kielze 意見が欲しいです🙏