Firstly, thanks for considering contributing to vault-secrets-operator. To make it a really great tool we need your help.
vault-secrets-operator is Apache 2.0 licenced and accepts contributions via GitHub pull requests. There are many ways to contribute, from writing tutorials or blog posts, improving the documentation, submitting bug reports and feature requests or writing code.
By contributing to this project you agree to the Developer Certificate of Origin. This was created by the Linux Foundation and is a simple statement that you, as a contributor, have the legal right to make the contribution.
To signify that you agree to the DCO you must signoff all commits:
git commit --signoff
- Fork the repository on GitHub
- Read the README for getting started as a user and learn how/where to ask for help
- If you want to contribute as a developer, continue reading this document for further instructions
- Play with the project, submit bugs, submit pull requests!
This project is written in Go and uses Operator SDK. To be able to contribute you will need:
-
A working Go installation of Go >= 1.18. You can check the official installation guide.
-
Make sure that
$(go env GOPATH)/bin
is in your shell'sPATH
. You can do so by runningexport PATH="$(go env GOPATH)/bin:$PATH"
-
Fork this repository and clone it by running:
git clone [email protected]:<yourusername>/vault-secrets-operator.git
As the project uses modules its recommeneded that you NOT clone under the
GOPATH
.
- Install Operator SDK as described here
Make sure you can run the tests and build the binary.
make build-local
-
Look at the existing issues to see if there is anything you would like to work on. If don't see anything then feel free to create your own feature request.
-
If you are a new contributor then take a look at the issues marked with good first issue.
-
Make your code changes within a feature branch:
git checkout -b <feature-name>
-
Add yourself to humans.txt if this is your first contribution.
-
Try to commit changes in logical units with a commit message in this format. Remember to signoff your commits.
-
Don't forget to update the docs if relevent. The README or the docs folder is where docs usually live.
-
Make sure the tests pass and that there are no linting problems.
Push your changes to your fork and then create a pull request to origin. Where possible use the PR template.
You can mark a PR as wotk in progress by prefixing the title of your PR with WIP:
.
We would like to follow the Conventional Commits format for commit messsages. The full specification can be read here. The format is:
<type>[optional scope]: <description>
[optional body]
[optional footer]
Where <type>
is one of the following:
feat
- a new featurefix
- a bug fixchore
- changes to the build pocess, code generation or anything that doesn't match elsewheredocs
- documentation only changesstyle
- changes that don't affect the meaning of the code (i.e. code formatting)refactor
- a change that doesn't fix a feature or bugtest
- changes to tests only.
The scope
can be a pkg name but is optional.
The body
should include details of what changed and why. If there is a breaking change then the body
should start with the
following: BREAKING CHANGE
.
The footer should include any related github issue numbers.
An example:
feat: Added connector status command
A new command has been added to show the status of tasks within a connector. The
command will return the number of failed tasks as the exit code.
Fixes: #123
A tool like Commitizen can be used to help with formatting commit messages.