Contributions are welcome to the Container Analysis Project in many forms, and there's always plenty to do.
Search in the code for any TODO and please contribute!
If you are a user and you find a bug, please submit an issue. Please try to provide sufficient information for someone else to reproduce the issue. One of the project's maintainers should respond to your issue within 24 hours. If not, please bump the issue and request that it be reviewed.
Review the issues list and find something that interests you. You could also check the "help wanted" list. It is wise to start with something relatively straight forward and achievable. Usually there will be a comment in the issue that indicates whether someone has already self-assigned the issue. If no one has already taken it, then add a comment assigning the issue to yourself, eg.: I'll work on this issue.. Please be considerate and rescind the offer in comments if you cannot finish in a reasonable time, or add a comment saying that you are still actively working the issue if you need a little more time.
We are using the GitHub Flow process to manage code contributions. If you are unfamiliar, please review that link before proceeding.
To work on something, whether a new feature or a bugfix:
-
Create a fork (if you haven't already)
-
Clone it locally
git clone https://github.com/mtarsel/ContainerAnalysis.git
- Add the upstream repository as a remote
git remote add upstream https://github.com/mtarsel/ContainerAnalysis.git
- Create a branch
Create a descriptively-named branch off of your cloned fork (more details here)
cd ContainerAnalysis
git checkout -b issue-xxx
- Commit your code
Commit to that branch locally, and regularly push your work to the same branch on the server. Every commit must include a Signoff line. You must use the -s flag.
git commit -s
- Commit messages
Commit messages must have a short description no longer than 50 characters followed by a blank line and a longer, more descriptive message that includes reference to issue(s) being addressed so that they will be automatically closed on a merge e.g. Closes #1234 or Fixes #1234.
- Pull Request (PR)
When you need feedback or help, or you think the branch is ready for merging, open a pull request (make sure you have first successfully built and tested your changes.
Note: if your PR does not merge cleanly, use git rebase master in your feature branch to update your pull request rather than using git merge master.
- Any code changes that affect documentation should be accompanied by corresponding changes (or additions) to the documentation and tests. This will ensure that if the merged PR is reversed, all traces of the change will be reversed as well.
After your Pull Request (PR) has been reviewed and signed off, a maintainer will merge it into the master branch.
[PEP8] (https://www.python.org/dev/peps/pep-0008/)
Projects or sub-projects will be lead by a set of maintainers. The project's maintainers will, from time-to-time, consider adding a new maintainer. An existing maintainer will post a pull request to the MAINTAINERS.txt file. If a majority of the maintainers concur in the comments, the pull request is then merged and the individual becomes a maintainer.
We have tried to make it as easy as possible to make contributions. This applies to how we handle the legal aspects of contribution. We use the same Developer's Certificate of Origin 1.1 (DCO) approach that the Linux Kernel community uses to manage code contributions. We simply ask that when submitting a pull request, the developer must include a sign-off statement in the pull request description.
Here is an example Signed-off-by line, which indicates that the submitter accepts the DCO:
Signed-off-by: John Doe <[email protected]>