blob: 0b113c0d8e0cf42ddd7adbbbaf0ac6f65b370d63 [file] [log] [blame]
Bill Wendling7d623452015-03-18 13:36:07 -07001Want to contribute? Great! First, read this page (including the small print at the end).
2
Eli Benderskycd0aa2f2015-03-19 06:22:10 -07003Before you contribute
4---------------------
5
6Before we can use your code, you must sign the `Google Individual Contributor
7License Agreement
8<https://developers.google.com/open-source/cla/individual?csw=1>`_ (CLA), which
9you can do online. The CLA is necessary mainly because you own the
Bill Wendling7d623452015-03-18 13:36:07 -070010copyright to your changes, even after your contribution becomes part of our
11codebase, so we need your permission to use and distribute your code. We also
12need to be sure of various other thingsfor instance that you'll tell us if you
13know that your code infringes on other people's patents. You don't have to sign
14the CLA until after you've submitted your code for review and a member has
15approved it, but you must do it before we can put your code into our codebase.
16Before you start working on a larger contribution, you should get in touch with
17us first through the issue tracker with your idea so that we can help out and
18possibly guide you. Coordinating up front makes it much easier to avoid
19frustration later on.
20
Eli Benderskycd0aa2f2015-03-19 06:22:10 -070021Code reviews
22------------
23
Bill Wendling7d623452015-03-18 13:36:07 -070024All submissions, including submissions by project members, require review. We
25use Github pull requests for this purpose.
26
Eli Benderskycd0aa2f2015-03-19 06:22:10 -070027YAPF coding style
28-----------------
29
Bill Wendlingc7e9b782015-04-09 16:02:23 -040030YAPF follows the `Chromium Python Style Guide
31<https://www.chromium.org/chromium-os/python-style-guidelines>`_. It's the same
32as the Google Python Style guide with two exceptions:
Sam Clegg4357fa32015-04-08 12:21:46 -070033
34- 2 spaces for indentation rather than 4.
35- CamelCase for function and method names rather than words_with_underscores.
36
Bill Wendlingcfcf45f2017-10-05 15:54:09 -070037The rationale for this is that YAPF was initially developed at Google where
38these two exceptions are still part of the internal Python style guide.
Eli Benderskycd0aa2f2015-03-19 06:22:10 -070039
40Small print
41-----------
42
Bill Wendling7d623452015-03-18 13:36:07 -070043Contributions made by corporations are covered by a different agreement than
44the one above, the Software Grant and Corporate Contributor License Agreement.