-
-
Notifications
You must be signed in to change notification settings - Fork 7.6k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Showing
1 changed file
with
23 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
# 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. | ||
|
||
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). | ||
|
||
## Code Contribution Guideline | ||
|
||
We welcome your contributions. | ||
To make the 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 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*. | ||
* 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. |
bd0176a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @anthonyfok,
did you left the original paragraphs intentionally redundant in the README?
bd0176a
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @digitalcraftsman,
Yes, I did left the original paragraphs in README.md, mainly due to my own laziness. :-)
The original paragraphs in README.md probably should be replaced by a reference to CONTRIBUTING.md, though I prefer leaving that decision to others. :-)
Cheers,
Anthony