Skip to content

Latest commit

 

History

History
60 lines (46 loc) · 2.33 KB

CONTRIBUTING.rst

File metadata and controls

60 lines (46 loc) · 2.33 KB

Contributing Code

A good pull request:

  • Is clear.
  • Works across all supported versions of Python.
  • Follows the existing style of the code base (see Codestyle section).
  • Has comments included as needed.
  • A test case that demonstrates the previous flaw that now passes with the included patch, or demonstrates the newly added feature.
  • If it adds/changes a public API, it must also include documentation for those changes.
  • Must be appropriately licensed (Apache 2.0).

Reporting An Issue/Feature

First, check to see if there's an existing issue/pull requests for the bug/feature.

If there isn't an existing issue there, please file an issue first. The ideal report includes:

  • A description of the problem/suggestion.
  • How to recreate the bug.
  • If relevant, including the versions of your:
    • Python interpreter
    • s3transfer
    • Optionally of the other dependencies involved (e.g. Botocore)
  • If possible, create a pull request with a (failing) test case demonstrating what's wrong. This makes the process for fixing bugs quicker & gets issues resolved sooner.

Codestyle

This project uses flake8 to enforce codstyle requirements. We've codified this process using a tool called pre-commit. pre-commit allows us to specify a config file with all tools required for code linting, and surfaces either a git commit hook, or single command, for enforcing these.

To validate your PR prior to publishing, you can use the following installation guide to setup pre-commit.

If you don't want to use the git commit hook, you can run the below command to automatically perform the codestyle validation:

$ pre-commit run

This will automatically perform simple updates (such as white space clean up) and provide a list of any failing flake8 checks. After these are addressed, you can commit the changes prior to publishing the PR. These checks are also included in our CI setup under the "Lint" workflow which will provide output on Github for anything missed locally.

See the flake8 section of the setup.cfg for the currently enforced rules.