From 373ca66287eb66c76832cf62159efbf58138353b Mon Sep 17 00:00:00 2001 From: Cameron Moore Date: Sat, 12 Mar 2016 17:35:06 -0600 Subject: [PATCH] hugo: Update contribution guidelines Refactor the contribution guidelines in the README and CONTRIBUTING files. Simplify the contribution guide in the README and move most of the complex stuff into CONTRIBUTING. Add an explicit commit message guidelines section to CONTRIBUTING. Keep all of the guidelines from Chris Beams except for the 72 character line limit (we don't follow that, nor does the Go team). Add three new guidelines: package prefix in subject, references in body, and encouragement of message body in general. Add a new section to CONTRIBUTING on using Git Remotes. --- CONTRIBUTING.md | 159 ++++++++++++++++++++++++++++++++++++++++++++---- README.md | 104 ++++++++----------------------- 2 files changed, 172 insertions(+), 91 deletions(-) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index f1c060cae..c25e63b39 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -1,23 +1,156 @@ # Contributing to Hugo -We welcome contributions to Hugo of any kind including documentation, themes, organization, tutorials, blog posts, bug reports, issues, feature requests, feature implementation, pull requests, answering questions on the forum, helping to manage issues, etc. +We welcome contributions to Hugo of any kind including documentation, themes, +organization, tutorials, blog posts, bug reports, issues, feature requests, +feature implementations, pull requests, answering questions on the forum, +helping to manage issues, etc. -The Hugo community and maintainers are very active and helpful and the project benefits greatly from this activity. +The Hugo community and maintainers are very active and helpful, and the project benefits greatly from this activity. [![Throughput Graph](https://graphs.waffle.io/spf13/hugo/throughput.svg)](https://waffle.io/spf13/hugo/metrics) -If you have any questions about how to contribute or what to contribute, please ask on the [forum](http://discuss.gohugo.io). +## Table of Contents -## Code Contribution Guideline +* [Asking Support Questions](#asking-support-questions) +* [Reporting Issues](#reporting-issues) +* [Submitting Patches](#submitting-patches) + * [Code Contribution Guidelines](#code-contribution-guidelines) + * [Git Commit Message Guidelines](#git-commit-message-guidelines) + * [Using Git Remotes](#using-git-remotes) + * [Build Hugo with Your Changes](#build-hugo-with-your-changes) + * [Add Compile Information to Hugo](#add-compile-information-to-hugo) -We welcome your contributions. -To make the process as seamless as possible, we ask for the following: +## Asking Support Questions -* Go ahead and fork the project and make your changes. We encourage pull requests to discuss code changes. +We have an active [discussion forum](http://discuss.gohugo.io) where users and developers can ask questions. +Please don't use the Github issue tracker to ask questions. + +## Reporting Issues + +If you believe you have found a defect in Hugo or its documentation, use +the Github [issue tracker](https://github.com/spf13/hugo/issues) to report the problem to the Hugo maintainers. +If you're not sure if it's a bug or not, start by asking in the [discussion forum](http://discuss.gohugo.io). +When reporting the issue, please provide the version of Hugo in use (`hugo version`) and your operating system. + +## Submitting Patches + +The Hugo project welcomes all contributors and contributions regardless of skill or experience level. +If you are interested in helping with the project, we will help you with your contribution. +Hugo is a very active project with many contributions happening daily. +Because we want to create the best possible product for our users and the best contribution experience for our developers, +we have a set of guidelines which ensure that all contributions are acceptable. +The guidelines are not intended as a filter or barrier to participation. +If you are unfamiliar with the contribution process, the Hugo team will help you and teach you how to bring your contribution in accordance with the guidelines. + +### Code Contribution Guidelines + +To make the contribution process as seamless as possible, we ask for the following: + +* Go ahead and fork the project and make your changes. We encourage pull requests to allow for review and discussion of code changes. * When you’re ready to create a pull request, be sure to: - * Sign the [CLA](https://cla-assistant.io/spf13/hugo) - * 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` - * Squash your commits into a single commit. `git rebase -i`. It’s okay to force update your pull request. - * **Write a good commit message.** 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*. Sometimes it makes sense to prefix the commit message with the packagename (or docs folder) all lowercased ending with a colon. That is fine, but the rest of the rules above apply. So it is "tpl: Add emojify template func", not "tpl: add emojify template func.", and "docs: Document emoji", not "doc: document emoji." - * Make sure `go test ./...` passes, and `go build` completes. Our [Travis CI loop](https://travis-ci.org/spf13/hugo) (Linux and OS X) and [AppVeyor](https://ci.appveyor.com/project/spf13/hugo/branch/master) (Windows) will catch most things that are missing. + * Sign the [CLA](https://cla-assistant.io/spf13/hugo). + * Have test cases for the new code. If you have questions about how to do this, please ask in your pull request. + * Run `go fmt`. + * Add documentation if you are adding new features or changing functionality. The docs site lives in `/docs`. + * Squash your commits into a single commit. `git rebase -i`. It’s okay to force update your pull request with `git push -f`. + * Make sure `go test ./...` passes, and `go build` completes. [Travis CI](https://travis-ci.org/spf13/hugo) (Linux and OS X) and [AppVeyor](https://ci.appveyor.com/project/spf13/hugo/branch/master) (Windows) will catch most things that are missing. + * Follow the **Git Commit Message Guidelines** below. + +### Git Commit Message Guidelines + +Quality Git commit messages are important in a large project to keep everyone informed; +therefore, we've established the following guidelines: + +1. Prefix the subject with the primary affected **package**. +1. After the package prefix, **capitalize** the subject. +1. End the subject **without punctuation**. +1. Use the **imperative** mood in the subject. +1. Limit the subject line to **50** characters. +1. Separate subject from body with a **blank line**. +1. Use the body to explain **what** and **why** instead of **how**. +1. If there is a helpful **reference** like a Github issue, mention it in the body + (ie. "Fixes #123" or "See #123"). +1. A message **body** is often desirable unless the code changes are trivial. + +To understand the rationales for many of these guidelines, +read [How to Write a Git Commit Message](http://chris.beams.io/posts/git-commit) by Chris Beams. + +An example: + +```text +tpl: Add custom index function + +Add a custom index template function that deviates from the stdlib simply by not +returning an "index out of range" error if an array, slice or string index is +out of range. Instead, we just return nil values. This should help make the +new default function more useful for Hugo users. + +Fixes #1949 +``` + +### Using Git Remotes + +Due to the way Go handles package imports, the best approach for working on a +Hugo fork is to use Git Remotes. Here's a simple walk-through for getting +started: + +1. Get the latest Hugo sources: + + ``` + go get -u -t github.com/spf13/hugo + ``` + +1. Change to the Hugo source directory: + + ``` + cd $GOPATH/src/github.com/spf13/hugo + ``` + +1. Create a new branch for your changes (the branch name is arbitrary): + + ``` + git checkout -b iss1234 + ``` + +1. After making your changes, commit them to your new branch: + + ``` + git commit -a -v + ``` + +1. Fork Hugo in Github. + +1. Add your fork as a new remote (the remote name, "fork" in this example, is arbitrary): + + ``` + git remote add fork git://github.com/USERNAME/hugo.git + ``` + +1. Push the changes to your new remote: + + ``` + git push --set-upstream fork iss1234 + ``` + +1. You're now ready to submit a PR based upon the new branch in your forked repository. + +### Build Hugo with Your Changes + +```bash +cd $GOPATH/src/github.com/spf13/hugo +go build +mv hugo /usr/local/bin/ +``` + +### Add Compile Information to Hugo + +To add compile information to Hugo, replace the `go build` command with the following *(replace `/path/to/hugo` with the actual path)*: + + go build -ldflags "-X /path/to/hugo/hugolib.CommitHash=`git rev-parse --short HEAD 2>/dev/null` -X github.com/spf13/hugo/hugolib.BuildDate=`date +%FT%T%z`" + +This will result in `hugo version` output that looks similar to: + + Hugo Static Site Generator v0.13-DEV-8042E77 buildDate: 2014-12-25T03:25:57-07:00 + +Alternatively, just run `make` — all the “magic” above is already in the `Makefile`. :wink: + diff --git a/README.md b/README.md index 536637fac..b223d58f6 100644 --- a/README.md +++ b/README.md @@ -7,6 +7,7 @@ A Fast and Flexible Static Site Generator built with love by [spf13](http://spf1 [Dev Chat](https://gitter.im/spf13/hugo) | [Documentation](https://gohugo.io/overview/introduction/) | [Installation Guide](https://gohugo.io/overview/installing/) | +[Contribution Guide](CONTRIBUTING.md) | [Twitter](http://twitter.com/spf13) [![GoDoc](https://godoc.org/github.com/spf13/hugo?status.svg)](https://godoc.org/github.com/spf13/hugo) @@ -52,42 +53,6 @@ Building the binaries is an easy task for an experienced `go` getter. Use the [installation instructions in the Hugo documentation](https://gohugo.io/overview/installing/). -### Clone the Hugo Project (Contributor) - -1. Make sure your local environment has the following software installed: - - * [Git](https://git-scm.com/) - * [Go][] 1.5+ - -2. [Fork the Hugo project on GitHub](https://github.com/spf13/hugo). - -3. Clone your fork: - - git clone https://github.com/YOURNAME/hugo - -4. Change into the `hugo` directory: - - cd hugo - -5. Install the Hugo project’s package dependencies: - - go get -u -v github.com/spf13/hugo - -6. Install the test dependencies (needed if you want to run tests): - - go get -v -t -d ./... - -7. Use a symbolic link to add your locally cloned Hugo repository to your `$GOPATH`, assuming you prefer doing development work outside of `$GOPATH`: - - ```bash - rm -rf "$GOPATH/src/github.com/spf13/hugo" - ln -s `pwd` "$GOPATH/src/github.com/spf13/hugo" - ``` - - Go expects all of your libraries to be found in`$GOPATH`. - -You can also find a [detailed guide](https://www.gohugo.io/tutorials/how-to-contribute-to-hugo/) in our documentation. - ### Build and Install the Binaries from Source (Advanced Install) Add Hugo and its package dependencies to your go `src` directory. @@ -100,59 +65,42 @@ To update Hugo’s dependencies, use `go get` with the `-u` option. go get -u -v github.com/spf13/hugo -## Contribute to Hugo +## Contributing to Hugo -We welcome contributions to Hugo of any kind including documentation, themes, organization, tutorials, blog posts, bug reports, issues, feature requests, feature implementation, pull requests, answering questions on the forum, helping to manage issues, etc. +For a complete guide to contributing to Hugo, see the [Contribution Guide](CONTRIBUTING.md). -The Hugo community and maintainers are very active and helpful and the project benefits greatly from this activity. +We welcome contributions to Hugo of any kind including documentation, themes, +organization, tutorials, blog posts, bug reports, issues, feature requests, +feature implementations, pull requests, answering questions on the forum, +helping to manage issues, etc. + +The Hugo community and maintainers are very active and helpful, and the project benefits greatly from this activity. [![Throughput Graph](https://graphs.waffle.io/spf13/hugo/throughput.svg)](https://waffle.io/spf13/hugo/metrics) -If you have any questions about how to contribute or what to contribute, please ask on the [forum](https://discuss.gohugo.io). +### Asking Support Questions -## Code Contribution Guideline +We have an active [discussion forum](http://discuss.gohugo.io) where users and developers can ask questions. +Please don't use the Github issue tracker to ask questions. -We welcome your contributions. -To make the process as seamless as possible, we ask for the following: +### Reporting Issues -* Go ahead and fork the project and make your changes. We encourage pull requests to discuss code changes. -* When you’re ready to create a pull request, be sure to: - * Sign the [CLA](https://cla-assistant.io/spf13/hugo) - * 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` - * Squash your commits into a single commit. `git rebase -i`. It’s okay to force update your pull request. - * **Write a good commit message.** 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*. Sometimes it makes sense to prefix the commit message with the packagename (or docs folder) all lowercased ending with a colon. That is fine, but the rest of the rules above apply. So it is "tpl: Add emojify template func", not "tpl: add emojify template func.", and "docs: Document emoji", not "doc: document emoji." - * Make sure `go test ./...` passes, and `go build` completes. Our [Travis CI loop](https://travis-ci.org/spf13/hugo) (Linux and OS X) and [AppVeyor](https://ci.appveyor.com/project/spf13/hugo/branch/master) (Windows) will catch most things that are missing. +If you believe you have found a defect in Hugo or its documentation, use +the Github issue tracker to report the problem to the Hugo maintainers. +If you're not sure if it's a bug or not, start by asking in the [discussion forum](http://discuss.gohugo.io). +When reporting the issue, please provide the version of Hugo in use (`hugo version`). -### Build Hugo with Your Changes +### Submitting Patches -```bash -cd /path/to/hugo -go build -o hugo main.go -mv hugo /usr/local/bin/ -``` +The Hugo project welcomes all contributors and contributions regardless of skill or experience level. +If you are interested in helping with the project, we will help you with your contribution. +Hugo is a very active project with many contributions happening daily. +Because we want to create the best possible product for our users and the best contribution experience for our developers, +we have a set of guidelines which ensure that all contributions are acceptable. +The guidelines are not intended as a filter or barrier to participation. +If you are unfamiliar with the contribution process, the Hugo team will help you and teach you how to bring your contribution in accordance with the guidelines. -### Add Compile Information to Hugo - -To add compile information to Hugo, replace the `go build` command with the following *(replace `/path/to/hugo` with the actual path)*: - - go build -ldflags "-X /path/to/hugo/hugolib.CommitHash=`git rev-parse --short HEAD 2>/dev/null` -X github.com/spf13/hugo/hugolib.BuildDate=`date +%FT%T%z`" - -This will result in `hugo version` output that looks similar to: - - Hugo Static Site Generator v0.13-DEV-8042E77 buildDate: 2014-12-25T03:25:57-07:00 - -Alternatively, just run `make` — all the “magic” above is already in the `Makefile`. :wink: - -### Run Hugo - -```bash -cd /path/to/hugo -go install github.com/spf13/hugo/hugolib -go run main.go -``` - -**Complete documentation is available at [Hugo Documentation][].** +For a complete guide to contributing code to Hugo, see the [Contribution Guide](CONTRIBUTING.md). [![Analytics](https://ga-beacon.appspot.com/UA-7131036-6/hugo/readme)](https://github.com/igrigorik/ga-beacon) [![Bitdeli Badge](https://d2weczhvl823v0.cloudfront.net/spf13/hugo/trend.png)](https://bitdeli.com/free "Bitdeli Badge")