Delta Between Two Patch Sets: flake8-eyeo/README.md
Issue 29602819:
Issue 5844 - Detect (more) redundant parentheses (Closed)
Base URL: https://hg.adblockplus.org/codingtools/
Left Patch Set: Fix format errors on test case
Created Dec. 7, 2017, 1:50 a.m.
Right Patch Set: Updated flake8-eyeo/README.md and applied patches to existing codebase
Created Jan. 25, 2018, 7:34 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« no previous file
|
flake8-eyeo/flake8_eyeo.py » ('j')
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| (no file at all) | |
| 1 # flake8-eyeo | 1 # flake8-eyeo |
| 2 | 2 |
| 3 A [flake8](https://flake8.readthedocs.io) extension that checks for compliance | 3 A [flake8](https://flake8.readthedocs.io) extension that checks for compliance |
| 4 with the | 4 with the |
| 5 [Adblock Plus coding style guide](https://adblockplus.org/coding-style#python) | 5 [Adblock Plus coding style guide](https://adblockplus.org/coding-style#python) |
| 6 which is used for all eyeo projects, and some bad practices which flake8 doesn't | 6 which is used for all eyeo projects, and some bad practices which flake8 doesn't |
| 7 handle by default. | 7 handle by default. |
| 8 | 8 |
| 9 | 9 |
| 10 ## Installation | 10 ## Installation |
| (...skipping 15 matching lines...) Expand all Loading... | |
| 26 * `A102`: Membership check on a tuple or list literal; use a set here | 26 * `A102`: Membership check on a tuple or list literal; use a set here |
| 27 * `A103`: Yoda condition | 27 * `A103`: Yoda condition |
| 28 * `A104`: `map()` or `filter()` called with lambda function | 28 * `A104`: `map()` or `filter()` called with lambda function |
| 29 * `A105`: Type called with literal or comprehension where a | 29 * `A105`: Type called with literal or comprehension where a |
| 30 literal/comprehension of that type can be used directly | 30 literal/comprehension of that type can be used directly |
| 31 * `A106`: Use augment assignment | 31 * `A106`: Use augment assignment |
| 32 * `A107`: `%` operator used for string formatting; this mechanism | 32 * `A107`: `%` operator used for string formatting; this mechanism |
| 33 has been superseded by the string's `format()` method | 33 has been superseded by the string's `format()` method |
| 34 * `A108`: `+` operator to concatenate more than two strings | 34 * `A108`: `+` operator to concatenate more than two strings |
| 35 * `A110`: Write single-line string literals as represented by `repr()` | 35 * `A110`: Write single-line string literals as represented by `repr()` |
| 36 * `A111`: Redundant parantheses around if or while condition | 36 * `A111`: Redundant parantheses |
| 37 * `A112`: Use `from __future__ import unicode_literals` instead of | 37 * `A112`: Use `from __future__ import unicode_literals` instead of |
| 38 prefixing literals with "u" | 38 prefixing literals with "u" |
| 39 | 39 |
| 40 | 40 |
| 41 ### Redundancy and complexity | 41 ### Redundancy and complexity |
| 42 | 42 |
| 43 * `A201`: Redundant or superfluos `global` or `nonlocal` declaration | 43 * `A201`: Redundant or superfluos `global` or `nonlocal` declaration |
| 44 * `A202`: Dead code after block is left | 44 * `A202`: Dead code after block is left |
| 45 * `A303`: Unused expression | 45 * `A303`: Unused expression |
| 46 * `A204`: Redundant or superfluos pass statement | 46 * `A204`: Redundant or superfluos pass statement |
| 47 * `A205`: Superfluos empty block | 47 * `A205`: Superfluos empty block |
| 48 * `A206`: Extraneous `else` statement after block is left | 48 * `A206`: Extraneous `else` statement after block is left |
| 49 * `A207`: Duplicate key in dict or set | 49 * `A207`: Duplicate key in dict or set |
| 50 | 50 |
| 51 | 51 |
| 52 ### Error-prone practices | 52 ### Error-prone practices |
| 53 | 53 |
| 54 * `A301`: Discouraged APIs | 54 * `A301`: Discouraged APIs |
| 55 * `A302`: Redefinition of built-in name | 55 * `A302`: Redefinition of built-in name |
| 56 * `A303`: Non-default source file encoding | 56 * `A303`: Non-default source file encoding |
| LEFT | RIGHT |
« no previous file
|
flake8-eyeo/flake8_eyeo.py » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
