Welcome and thank you for making it this far and considering contributing to stackit-cert-manager-webhook. We always appreciate any contributions by raising issues, improving the documentation, fixing bugs or adding new features.
Before opening a PR please read through this document. If you want to contribute but don't know how to start or have any questions feel free to reach out to us on Github Discussions. Answering any questions or discussions there is also a great way to contribute to the community.
Please keep in mind to open an issue whenever you plan to make an addition to features to discuss it before implementing it.
To contribute any code to this repository just do the following:
- Make sure you have Go's latest version installed
- Fork this repository
- Run
make build
to make sure everything's setup correctly - Make your changes
Please follow the seven rules of greate Git commit messages and make sure to keep your commits clean and atomic. Your PR won't be squashed before merging so the commits should tell a story.
Optional: Sign-off on all Git commits by running
git commit -s
. Take a look at the Gihub Docs for further information.Add documentation and tests for your addition if needed.
- Run
make lint test
to ensure your code is ready to be mergedIf any linting issues occur please fix them. Using a nolint directive should only be used as a last resort.
- Open a PR and make sure the CI pipelines succeed.
Your PR needs to have a semantic title, which can look like:
type(scope) Short Description
All availablescopes
&types
are defined in semantic.yml - Wait for one of the maintainers to review your code and react to the comments.
- After approval merge the PR
- Thank you for your contribution! :)