2015-05-06 21:18:46 +00:00
|
|
|
This document describes the way you can contribute to the restic project.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
Ways to Help Out
|
2015-05-06 07:11:07 +00:00
|
|
|
================
|
|
|
|
|
|
|
|
Thank you for your contribution!
|
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
There are several ways you can help us out. First of all code contributions and
|
|
|
|
bug fixes are most welcome. However even "minor" details as fixing spelling
|
|
|
|
errors, improving documentation or pointing out usability issues are a great
|
|
|
|
help also.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
The restic project uses the GitHub infrastructure (see the
|
|
|
|
[project page](https://github.com/restic/restic)) for all related discussions
|
|
|
|
as well as the `#restic` channel on `irc.freenode.net`.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
If you want to find an area that currently needs improving have a look at the
|
|
|
|
open issues listed at the
|
|
|
|
[issues page](https://github.com/restic/restic/issues). This is also the place
|
|
|
|
for discussing enhancement to the restic tools.
|
|
|
|
|
|
|
|
If you are unsure what to do, please have a look at the issues, especially
|
|
|
|
those tagged
|
|
|
|
[minor complexity](https://github.com/restic/restic/labels/minor%20complexity).
|
|
|
|
|
|
|
|
Providing Patches
|
2015-05-06 07:11:07 +00:00
|
|
|
=================
|
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
You have fixed an annoying bug or have added a new feature? Very cool! Let's
|
|
|
|
get it into the project! First we would kindly ask you to fork our project on
|
|
|
|
GitHub if you haven't done so already.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
The restic project uses the `gofmt` tool for go source indentation, so please
|
|
|
|
run
|
2015-05-06 07:11:07 +00:00
|
|
|
|
|
|
|
gofmt -w **/*.go
|
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
in the project root directory before committing. Installing the script
|
|
|
|
`fmt-check` from https://github.com/edsrzf/gofmt-git-hook locally as a
|
|
|
|
pre-commit hook checks formatting before committing automatically, just copy
|
|
|
|
this script to `.git/hooks/pre-commit`.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
Then commit your changes as fine grained as possible, as smaller patches, that
|
|
|
|
handle one and only one issue are easier to discuss and merge.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
Please provide the patches for each bug or feature in a separate branch and
|
|
|
|
open up a pull request for each.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
Code Review
|
2015-05-06 07:11:07 +00:00
|
|
|
===========
|
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
The restic project encourages actively reviewing the code, as it will store
|
|
|
|
your precious data, so it's not uncommon to receive comments on provided
|
|
|
|
patches.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
If you are reviewing other contributor's code please consider the following
|
|
|
|
when reviewing:
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
* Be nice. Please make the review comment as constructive as possible so all
|
|
|
|
participants will learn something from your review.
|
2015-05-06 07:11:07 +00:00
|
|
|
|
2015-05-06 21:18:46 +00:00
|
|
|
As a contributor you might be asked to rewrite portions of your code to make it
|
|
|
|
fit better into the upstream sources.
|