Contributions are welcome via GitHub Pull Requests. This document outlines the process to help get your contribution accepted.
Any type of contribution is welcome: new features, bug fixes, documentation improvements, etc.
- Fork this repository, develop, and test your changes.
- Submit a pull request.
When submitting a PR make sure that:
- It must pass CI jobs for linting and test the changes (if any).
- It must follow container best practices.
- The title of the PR is clear enough and starts with "[bitnami/]"
- If necessary, add information to the repository's
README.md
.
The sign-off is a simple line at the end of the explanation for a commit. All commits needs to be signed. Your signature certifies that you wrote the patch or otherwise have the right to contribute the material. The rules are pretty simple, you only need to certify the guidelines from developercertificate.org.
Then you just add a line to every git commit message:
Signed-off-by: Joe Smith <joe.smith@example.com>
Use your real name (sorry, no pseudonyms or anonymous contributions.)
If you set your user.name
and user.email
git configs, you can sign your commit automatically with git commit -s
.
Note: If your git config information is set properly then viewing the git log
information for your commit will look something like this:
Author: Joe Smith <joe.smith@example.com>
Date: Thu Feb 2 11:41:15 2018 -0800
Update README
Signed-off-by: Joe Smith <joe.smith@example.com>
Notice the Author
and Signed-off-by
lines match. If they don't your PR will be rejected by the automated DCO check.
- Changes are manually reviewed by Bitnami team members usually within a business day.
- Once the changes are accepted, the PR is tested (if needed) into the Bitnami CI pipeline, the container is deployed and tested (verification and functional tests) using docker-compose and Helm (if there is an associated Helm Chart).
- The PR is merged by the reviewer(s) in the GitHub
master
branch. - Then our CI/CD system is going to push the container image to the different registries including the recently merged changes.
NOTE: Please note that, in terms of time, may be a slight difference between the appearance of the code in GitHub and the image with the changes in the different registries.