Welcome and thank you for making it this far and considering contributing to external-dns-selectel-webhook. We always appreciate any contributions by raising issues, improving the documentation, fixing bugs in the CLI or adding new features.
Before opening a PR please read through this document.
To contribute any code to this repository just do the following:
- Make sure you have a Go version installed that is similar to the version of this project
- 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.ymlAn example PR tile for adding a new feature for the CLI would look like:
cli(feat) Add saving output to file
- Wait for two of the maintainers to review your code and react to the comments.
- After two approvals merge the PR
- Thank you for your contribution! :)