We love your input! We want to make contributing to this project as easy and transparent as possible, whether it's:
- Reporting a bug
- Discussing the current state of the code
- Submitting a fix
- Proposing new features
- Becoming a maintainer
We use Github to host code, to track issues and feature requests, as well as accepting pull requests.
Pull requests are the best way to propose changes to the codebase (we use Github Flow). We actively welcome your pull requests:
- Fork the repo and create your branch from
master
. - If you've added code that should be tested, add tests.
- If you've changed APIs, update the documentation.
- Ensure the test suite passes.
- Make sure your code lints.
- Issue that pull request!
In short, when you submit code changes, your submissions are understood to be under the same Apache License 2.0 that covers the project. Feel free to contact the maintainers if that's a concern.
Report bugs using Github Issues
We use GitHub issues to track public bugs. Report a bug by opening a new issue; it's that easy!
Great Bug Reports tend to have:
- A quick summary and/or background
- Steps to reproduce
- Be specific!
- Give sample code if you can
- What you expected would happen
- What actually happens
- Notes (possibly including why you think this might be happening, or stuff you tried that didn't work)
People love thorough bug reports. I'm not even kidding.
We aim to follow PEP 8 recommendations in this project, with a max. line width of 100 characters. Source code documentation relies on restructured text, so that we can generate pretty API docs with Sphinx.
You may use flake8 to check for common style issues:
flake8 src --max-complexity=10 --max-line-length=100 --ignore=F401,W504
But please be aware that a consistent code style can only be achieved by reading and understanding code; tools like linters are useful, but they can never confirm that everything is ok.
This document was adapted from the open-source contribution guidelines for Facebook's Draft.