How to Contribute¶
The Helm Operator is Apache 2.0 licensed and accepts contributions via GitHub pull requests. This document outlines some of the conventions on development workflow, commit message formatting, contact points and other resources to make it easier to get your contribution accepted.
We gratefully welcome improvements to issues and documentation as well as to code.
Working on issues¶
If you like the operator and want to get involved in the project, a great way to get started is reviewing our needs validation issues.
The idea here is that new issues are confirmed, which might require asking
for more information, testing with a fresh environment. Once confirmed, the
blocked needs validation label is removed, and the issue can be worked on.
Please talk to us on Slack, if you should get stuck anywhere. We appreciate any help and look forward to talking to you soon!
Certificate of Origin¶
By contributing to this project you agree to the Developer Certificate of Origin (DCO). This document was created by the Linux Kernel community and is a simple statement that you, as a contributor, have the legal right to make the contribution. No action from you is required, but it's a good idea to see the DCO file for details before you start contributing code to Helm Operator.
The Flux developers use a mailing list to discuss development as well. Simply subscribe to flux-dev on cncf.io to join the conversation.
Agenda and minutes of previous Flux Dev meetings can be reviewed in this Google doc.
Meeting recordings can be viewed in this Youtube playlist.
- 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
- If you are a new contributor, the following two issue labels might be interesting to you:
- help wanted
- If you have questions, concerns, get stuck or need a hand, let us know on the Slack channel. We are happy to help and look forward to having you part of the team. No matter in which capacity.
- Play with the project, submit bugs, submit pull requests!
This is a rough outline of how to prepare a contribution:
- Create a topic branch from where you want to base your work (usually branched from master).
- Make commits of logical units.
- Make sure your commit messages are in the proper format (see below).
- Push your changes to a topic branch in your fork of the repository.
- If you changed code:
- add automated tests to cover your changes
- Submit a pull request to the original repository.
How to build and run the project¶
Refer to the Building guide to find out how to build from source.
Refer to the Get started developing guide for a walkthrough on developing Helm Operator locally.
How to run the test suite¶
You can run the unit tests by simply doing
These things will make a PR more likely to be accepted:
- a well-described requirement
- tests for new code
- tests for old code!
- new code and tests follow the conventions in old code and tests
- a good commit message (see below)
- All code must abide Go Code Review Comments
- Names should abide What's in a name
- Code must build on both Linux and Darwin, via plain
- Code should have appropriate test coverage and tests should be written
to work with
In general, we will merge a PR once one maintainer has endorsed it. For substantial changes, more people may become involved, and you might get asked to resubmit the PR or divide the changes into more than one PR.
Format of the Commit Message¶
We prefer the following rules for good commit messages:
- Limit the subject to 50 characters and write as the continuation of the sentence "If applied, this commit will ..."
- Explain what and why in the body, if more than a trivial change; wrap it at 72 characters.
The following article has some more helpful advice on documenting your work.