All members of the Garden Linux community must abide by the CNCF Code of Conduct. Only by respecting each other can we develop a productive, collaborative community. Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting [email protected] and/or a Garden Linux project maintainer.
Garden Linux uses GitHub to manage reviews of pull requests.
If you are a new contributor see: Steps to Contribute
If you have a trivial fix or improvement, go ahead and create a pull request.
If you plan to do something more involved, first discuss your ideas on our mailing list. This will avoid unnecessary work and surely give you and us a good deal of inspiration.
Should you wish to work on an issue, please claim it first by commenting on the GitHub issue that you want to work on it. This is to prevent duplicated efforts from contributors on the same issue.
If you have questions about one of the issues, with or without the tag, please comment on them and one of the maintainers will clarify it.
We kindly ask you to follow the Pull Request Checklist to ensure reviews can happen accordingly.
You are welcome to contribute code to Garden Linux in order to fix a bug or to implement a new feature.
The following rules govern code contributions:
- Contributions must be licensed under the MIT License.
- You need to sign the Developer Certificate of Origin.
You are welcome to contribute documentation and code to Garden Linux.
The following rules govern documentation contributions:
- Contributions must be licensed under the Creative Commons Attribution 4.0 International License.
- You need to sign the Developer Certificate of Origin.
Due to legal reasons, contributors will be asked to accept a Developer Certificate of Origin (DCO) before they submit the first pull request to this projects, this happens in an automated fashion during the submission process. We use the standard DCO text of the Linux Foundation.
By contributing to Garden Linux please make sure to take care of our git branch naming convention. All new branches should provide a prefix (mostly feature/
) followed by a reference to an issue and a summary.
Example:
feature/<issue-id>-<summary>
which could look like:
feature/1337-add-fancy-stuff
This example represents the prefix feature
, followed by the referenced GitHub issue 1337
with a summary.
Next to this, more specific prefixes (unlike only feature/
) may be used.
Prefix | Usage |
---|---|
feature/ | Adding new feature(s) (in code) to Garden Linux (generic) |
doc/ | Adding/Changing documentation in Garden Linux |
enhancement/ | Enhancements in Garden Linux that may change code, doc, features, etc. |
cleanup/ | Code cleanup (e.g. remove legacy/deprecated content |
ci/ | Changes related to our internal CI/CD pipeline (only used by Garden Linux members) |
archive/ | Stale branches that are need to keep (not used for new branches) |
release/ | Only used by Garden Linux members. By newer release used as a tag instead of branch |
-
Branch from the master branch and, if needed, rebase to the current master branch before submitting your pull request. If it doesn’t merge cleanly with master you may be asked to rebase your changes.
-
Commits should be as small as possible, while ensuring that each commit is correct independently (i.e., each commit should compile and pass tests).
-
Commit messages must follow the Conventional Commits specification so that we automatically produce meaningful release notes.
-
Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit and/or integration tests. If tested manually, provide information about the test scope in the PR description.
-
Create Work In Progress [WIP] pull requests only if you need a clarification or an explicit review before you can continue your work item.
-
If your patch is not getting reviewed or you need a specific person to review it, you can @-reply a reviewer asking for a review in the pull request or a comment, or you can ask for a review on our mailing list.
-
Post review:
- If a review requires you to change your commit(s), please test the changes again.
- Amend the affected commit(s) and force push onto your branch.
- Set respective comments in your GitHub review to resolved.
- Create a general PR comment to notify the reviewers that your amendments are ready for another round of review.
-
When merging into main, make sure that all commits belonging to a pull-request get squashed into one single commit that gets merged.
We use GitHub issues to track bugs and enhancement requests. Please provide as much context as possible when you open an issue. The information you provide must be comprehensive enough to reproduce that issue for the assignee. Therefore, contributors may use but aren’t restricted to the issue template provided by the Garden Linux maintainers.
The mailing list is hosted through Google Groups. To receive the lists' emails, join the group, as you would any other Google Groups at https://groups.google.com/g/gardenlinux.