rakkess
uses GitHub to manage reviews of pull requests.
-
If you have a trivial fix or improvement, go ahead and create a pull request.
-
Code must be properly formatted using
go fmt
-
If you plan to do something more involved, first raise an issue to discuss your idea. This will avoid unnecessary work.
-
Relevant coding style guidelines are the Go Code Review Comments and the Formatting and style section of Peter Bourgon's Go: Best Practices for Production Environments.
- Build via
make dev
to create the binary file./rakkess
. - Run unit tests with:
make test
- Run coverage with:
make coverage
-
Use the latest stable Go release
-
Branch from master and, if needed, rebase to the current master branch before submitting your pull request. If it doesn't merge cleanly with master you will be asked to rebase your changes.
-
Commits should be small units of work with one topic. Each commit should be correct independently.
-
Add tests relevant to the fixed bug or new feature.
-
Add a DCO /
Signed-off-by
line in any commit message (git commit --signoff
).
go env
shows helpful info about the current env setup for go.- Check here for more info on setting
$GOPATH
and$GOROOT
env vars.
mkdir -p $HOME/go/src
export GOPATH=$HOME/go
go get -u github.com/corneliusweig/rakkess
- Set
$GOROOT
depending on your OS and Go installation method:- MacOS, Go installed via brew:
export GOROOT=/usr/local/opt/go/libexec/
- MacOS, Go installed via brew:
- Now you should be able to build:
cd $GOPATH/src/github.com/corneliusweig/rakkess/
make dev
This is a checklist for new releases:
- Create release notes in
doc/releases
by runninghack/release_notes.sh
. - Update usage instructions, if applicable.
- Create a new tag via
hack/make_tag.sh
. - Push the tag to GitHub
git push origin v1.3.1
. - Check that PR is auto-created in krew-index.
- When the release is created, update its summary with the release notes.