diff options
author | Andrew Bonventre <andybons@golang.org> | 2018-02-20 19:14:00 -0500 |
---|---|---|
committer | Andrew Bonventre <andybons@golang.org> | 2018-02-21 02:07:46 +0000 |
commit | 16c5d30c84a7dbba88f89585b0238f52cc6b4494 (patch) | |
tree | 53685a26e250eb8acf733cebdac886471e0a802b /.github | |
parent | 2629703a5c36210435369e0dc24db6497ea059a8 (diff) | |
download | go-git-16c5d30c84a7dbba88f89585b0238f52cc6b4494.tar.gz |
github: update Pull Request template
+ Move from Markdown checklist to text. The first PR comment is
presented as text when creating it.
+ Add the note about Signed-Off-By: not being required.
Change-Id: I0650891dcf11ed7dd367007148730ba2917784fe
Reviewed-on: https://go-review.googlesource.com/95696
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Diffstat (limited to '.github')
-rw-r--r-- | .github/PULL_REQUEST_TEMPLATE | 25 | ||||
-rw-r--r-- | .github/PULL_REQUEST_TEMPLATE.md | 21 |
2 files changed, 25 insertions, 21 deletions
diff --git a/.github/PULL_REQUEST_TEMPLATE b/.github/PULL_REQUEST_TEMPLATE new file mode 100644 index 0000000000..2e978b6138 --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE @@ -0,0 +1,25 @@ +This PR will be imported into Gerrit with the title and first +comment (this text) used to generate the subject and body of +the Gerrit change. + +**Please ensure you adhere to every item in this list.** + +More info can be found at https://github.com/golang/go/wiki/CommitMessage + ++ The PR title is formatted as follows: `net/http: frob the quux before blarfing` + + The package name goes before the colon + + The part after the colon uses the verb tense + phrase that completes the blank in, + "This change modifies Go to ___________" + + Lowercase verb after the colon + + No trailing period + + Keep the title as short as possible. ideally under 76 characters or shorter ++ No Markdown ++ The first PR comment (this one) is wrapped at 76 characters, unless it's + really needed (ASCII art, table, or long link) ++ If there is a corresponding issue, add either `Fixes #1234` or `Updates #1234` + (the latter if this is not a complete fix) to this comment ++ If referring to a repo other than `golang/go` you can use the + `owner/repo#issue_number` syntax: `Fixes golang/tools#1234` ++ We do not use Signed-off-by lines in Go. Please don't add them. + Our Gerrit server & GitHub bots enforce CLA compliance instead. ++ Delete these instructions once you have read and applied them diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md deleted file mode 100644 index 85899b97ae..0000000000 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ /dev/null @@ -1,21 +0,0 @@ -This PR will be imported into Gerrit with the title and first -comment (this text) used to generate the subject and body of -the Gerrit change. -**Please ensure you adhere to and check off each item in this list.** -More info can be found at https://github.com/golang/go/wiki/CommitMessage - -+ [ ] The PR title is formatted as follows: `net/http: frob the quux before blarfing` - + The package name goes before the colon - + The part after the colon uses the verb tense + phrase that completes the blank in, - "This change modifies Go to ___________" - + Lowercase verb after the colon - + No trailing period - + Keep the title as short as possible. ideally under 76 characters or shorter -+ [ ] No Markdown -+ [ ] The first PR comment (this one) is wrapped at 76 characters, unless it's - really needed (ASCII art, table, or long link) -+ [ ] If there is a corresponding issue, add either `Fixes #1234` or `Updates #1234` - (the latter if this is not a complete fix) to this comment -+ [ ] If referring to a repo other than `golang/go` you can use the - `owner/repo#issue_number` syntax: `Fixes golang/tools#1234` -+ [ ] Delete these instructions once you have read and applied them |