Contributing to FoFiX¶
As an open source project, FoFiX welcomes contributions of many forms.
Please use the issue tracker on GitHub.
Be sure to include all relevant information (traceback, version, reproducing steps, …).
Patches are welcome either as pull requests on GitHub. Please avoid duplicated patches, and make small PR.
To avoid duplicated work:
- if there is no issue about your bug, create one
- tell people that you’re working on a patch.
Here are some guidelines:
fork the repo
create a topic branch based on
git checkout -b my-topic-branch master
push your branch:
git push origin my-topic-branch
submit a pull request to
Then, your pull request (PR) will be reviewed. This could take several days, which is also good for maturity.
After your pull request is merged, do not forget to:
- remove your topic branch
- update your
masterbranch with upstream version
- celebrate :).
Git Commit Guidelines¶
In order to make commit messages readable, they should follow some rules, based on community standards:
- separate subject from body with a blank line
- limit the subject line to 50 characters
- capitalize the subject line
- do not end the subject line with a period
- use the imperative mood in the subject line
- wrap the body at 72 characters
- use the body to explain what and why vs. how.
This will look like this:
Capitalized, short (50 chars or less) summary More detailed explanatory text, if necessary. Wrap it to about 72 characters or so. In some contexts, the first line is treated as the subject of an email and the rest of the text as the body. The blank line separating the summary from the body is critical (unless you omit the body entirely); tools like rebase can get confused if you run the two together. Write your commit message in the imperative: "Fix bug" and not "Fixed bug" or "Fixes bug." This convention matches up with commit messages generated by commands like git merge and git revert. Further paragraphs come after blank lines. - Bullet points are okay, too - Typically a hyphen or asterisk is used for the bullet, preceded by a single space, with blank lines in between, but conventions vary here - Use a hanging indent Fix #1234 Ref #2345
You can test your commit with gitlint before pushing your code.
For more information about those community standards, take a look at: