Before contributing to this repository, first please discuss the change you wish to make via an issue, or any other method of communication with the maintainers of this repository.
You can also search this project for issues with the following labels:
| Label | Search Term | Description |
|---|---|---|
| good-first-issue | is:issue is:open label:good-first-issue |
Recommended for first-time contributors! These are well-defined, and allow a user to get familiar with the project's workflow before tackling more complex issues. |
| help wanted | is:issue is:open label:"help wanted" |
General issues where contributors help is wanted. |
| question | is:issue is:open label:question |
Open discussions to resolve everything from implementation details to desired functionality. |
Public modules, functions, classes, and methods must be documented using Python docstrings. Non-public functions and methods must also be documented for defining the API contract. In addition to being useful for generating documentation, docstrings add clarity when looking through the source code or using the built-in help system, and can be leveraged in autocompletion by IDEs.
Please see PEP 257 for details on semantics and conventions associated with Python docstrings.
Docstrings must follow Google style docstrings. This docstring style is more pleasant to read when browsing the source.
All functions and methods should be type annotated. This allows for static analysis and more intelligent code completion from tools & IDEs.
Changes should always include tests. If this is a bug fix it is a good idea to add the tests as the first commit of the pull request and the changes to fix the issue in subsequent commits to make it easier to validate it.
Before opening a pull request to this repository, first please make sure there is a GitHub issue
where your change has been discussed with the maintainers. Mention the issue number in your pull
request description using one of the
supported keywords. For example,
adding Closes: #100 in the pull request description will link the PR with the issue and GitHub
will automatically close the issue upon merging it.
Do not include the issue reference in your commit messages however, add it only in the description of the PR.