mirror of
https://github.com/gohugoio/hugo.git
synced 2024-11-21 20:46:30 -05:00
Emphasize that commit message titles should start with a capital letter
This commit is contained in:
parent
619c16fd03
commit
2843605963
1 changed files with 1 additions and 1 deletions
|
@ -116,7 +116,7 @@ To make the process as seamless as possible, we ask for the following:
|
||||||
* Have test cases for the new code. If you have questions about how to do it, please ask in your pull request.
|
* Have test cases for the new code. If you have questions about how to do it, please ask in your pull request.
|
||||||
* Run `go fmt`
|
* Run `go fmt`
|
||||||
* Squash your commits into a single commit. `git rebase -i`. It’s okay to force update your pull request.
|
* Squash your commits into a single commit. `git rebase -i`. It’s okay to force update your pull request.
|
||||||
* This [blog article](http://chris.beams.io/posts/git-commit/) is a good resource for learning how to write good commit messages, the most important part being that each commit message should have a title/subject in imperative mode without trailing period: *"Return error on wrong use of the Paginator"*, **NOT** *"Returning some error."* Also, if your commit references one or more GitHub issues, always end your commit message body with *See #1234* or *Fixes #1234*. Replace *1234* with the GitHub issue ID. The last example will close the issue when the commit is merged into *master*.
|
* This [blog article](http://chris.beams.io/posts/git-commit/) is a good resource for learning how to write good commit messages, the most important part being that each commit message should have a title/subject in imperative mood starting with a capital letter and no trailing period: *"Return error on wrong use of the Paginator"*, **NOT** *"returning some error."* Also, if your commit references one or more GitHub issues, always end your commit message body with *See #1234* or *Fixes #1234*. Replace *1234* with the GitHub issue ID. The last example will close the issue when the commit is merged into *master*.
|
||||||
* Make sure `go test ./...` passes, and `go build` completes. Our [Travis CI loop](https://travis-ci.org/spf13/hugo) (Linux) and [AppVeyor](https://ci.appveyor.com/project/spf13/hugo/branch/master) (Windows) will catch most things that are missing.
|
* Make sure `go test ./...` passes, and `go build` completes. Our [Travis CI loop](https://travis-ci.org/spf13/hugo) (Linux) and [AppVeyor](https://ci.appveyor.com/project/spf13/hugo/branch/master) (Windows) will catch most things that are missing.
|
||||||
|
|
||||||
### Build Hugo with Your Changes
|
### Build Hugo with Your Changes
|
||||||
|
|
Loading…
Reference in a new issue