diff options
Diffstat (limited to 'SUBMITTING_PATCHES')
| -rw-r--r-- | SUBMITTING_PATCHES | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/SUBMITTING_PATCHES b/SUBMITTING_PATCHES index cba67416..50e2cf77 100644 --- a/SUBMITTING_PATCHES +++ b/SUBMITTING_PATCHES | |||
| @@ -2,6 +2,7 @@ Short Version: | |||
| 2 | 2 | ||
| 3 | - Make small logical changes. | 3 | - Make small logical changes. |
| 4 | - Provide a meaningful commit message. | 4 | - Provide a meaningful commit message. |
| 5 | - Check for coding errors with pylint | ||
| 5 | - Make sure all code is under the Apache License, 2.0. | 6 | - Make sure all code is under the Apache License, 2.0. |
| 6 | - Publish your changes for review: | 7 | - Publish your changes for review: |
| 7 | 8 | ||
| @@ -33,7 +34,14 @@ If your description starts to get too long, that's a sign that you | |||
| 33 | probably need to split up your commit to finer grained pieces. | 34 | probably need to split up your commit to finer grained pieces. |
| 34 | 35 | ||
| 35 | 36 | ||
| 36 | (2) Check the license | 37 | (2) Check for coding errors with pylint |
| 38 | |||
| 39 | Run pylint on changed modules using the provided configuration: | ||
| 40 | |||
| 41 | pylint --rcfile=.pylintrc file.py | ||
| 42 | |||
| 43 | |||
| 44 | (3) Check the license | ||
| 37 | 45 | ||
| 38 | repo is licensed under the Apache License, 2.0. | 46 | repo is licensed under the Apache License, 2.0. |
| 39 | 47 | ||
| @@ -49,7 +57,7 @@ your patch. It is virtually impossible to remove a patch once it | |||
| 49 | has been applied and pushed out. | 57 | has been applied and pushed out. |
| 50 | 58 | ||
| 51 | 59 | ||
| 52 | (3) Sending your patches. | 60 | (4) Sending your patches. |
| 53 | 61 | ||
| 54 | Do not email your patches to anyone. | 62 | Do not email your patches to anyone. |
| 55 | 63 | ||
