Contributions of all kinds are welcome. In particular pull requests are appreciated. The authors and maintainers will endeavor to help walk you through any issues in the pull request discussion, so please feel free to open a pull request even if you are new to such things.
Our company values guide us in our day-to-day interactions and decision-making. Our open source projects are no exception and they will define the standards for how to engage with the project through a code of conduct.
Please, make sure you read both of them before contributing, so you can help us to maintain a healthy community.
Before you ask a question, it is best to search for existing Issues that might help you. In case you have found a suitable issue and still need clarification, you can write your question in this issue. It is also advisable to search the internet for answers first.
If you then still feel the need to ask a question and need clarification, we recommend the following:
- Open an Issue.
- Provide as much context as you can about what you’re running into.
- Provide project and platform versions (golang, operator-sdk, etc), depending on what seems relevant.
- The community will then take care of the issue as soon as possible.
We use GitHub issues to track bugs and errors. If you run into an issue with the project:
- Open an Issue.
- Explain the behavior you would expect and the actual behavior.
- Please provide as much context as possible and describe the reproduction steps that someone else can follow to recreate the issue on their own. This usually includes your code. For good bug reports you should isolate the problem and create a reduced test case.
Once it’s filed:
- The project team will label the issue accordingly.
- A team member will try to reproduce the issue with your provided steps. If there are no reproduction steps or no obvious way to reproduce the issue, the team will ask you for those steps and mark the issue as
needs-reproducer
. Bugs with this tag will not be addressed until they are reproduced. - If the team is able to reproduce the issue, it will be marked
needs-fix
and left to be implemented by someone. Other labels can be used in addition to better describe the issue or its criticality.
Enhancement suggestions are tracked as GitHub issues.
- Use a clear and descriptive title for the issue to identify the suggestion.
- Provide a step-by-step description of the suggested enhancement in as many details as possible.
- Describe the current behavior, the expected one, and why you expect this behavior. At this point you can also list which alternatives do not work for you.
- Explain why this enhancement would be useful to other users. You may also want to point out the other projects that solved it better and could serve as inspiration.
Before contributing code or documentation to this project, make sure you read the following sections.
The project follows the Conventional Commits specification and enforces it using gitlint. The rules for this project are specified in the .gitlint config file. There is also a second rule file for the commit description that can be found in the .github/gitlint directory.
The commit message should contain an overall explanation about the change and the motivation behind it. Please note that mentioning a Jira ticket ID or a GitHub issue, isn't a replacement for that.
A well formatted commit would look something like this:
feat(issue-id): what this commit does
Overall explanation of what this commit is achieving and the motivation behind it.
Signed-off-by: Your Name <[email protected]>
This project also enforces GPG signed commits. More information on commit signing and how to do it can be found in the git official docs or in this GitHub guide. You can see whether or not the commit was successfully signed by the Verified
bubble next to your commit in the GitHub UI or using git log
.
All changes must come from a pull request (PR) and cannot be directly committed. While anyone can engage in activity on a PR, pull requests are only approved by team members.
Before a pull request can be merged:
- The content of the PR has to be relevant to the PR itself
- The contribution must follow the style guidelines of this project
- Multiple commits should be used if the PR is complex and clarity can be improved, but they should still relate to a single topic
- For code contributions, tests have to be added/modified to ensure the code works
- There has to be at least one approval
- The feature branch must be rebased so it contains the latest changes from the target branch
- The CI has to pass successfully
- Every comment has to be addressed and resolved