Unified Diff: flake8-abp/tests/A111.py
Issue 29340727:
Noissue - Added flake8 extension accounting for our coding style and some other stuff (Closed)
Patch Set: Addressed comments, fixed two bugs, use ascii()
Created May 9, 2016, 4:47 p.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« flake8-abp/tests/A110.py ('k')
|
flake8-abp/tests/A201.py » ('j')
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: flake8-abp/tests/A111.py |
| =================================================================== |
| new file mode 100644 |
| --- /dev/null |
| +++ b/flake8-abp/tests/A111.py |
| @@ -0,0 +1,27 @@ |
| +def redundant_parenthesis(x, y, z): |
| + # * A111 |
| + while (x): |
| + # * A111 |
| + if ((x or y) and z): |
| + pass |
| + # * A111 |
| + elif (x == max(y, z)): |
| + pass |
| + else: |
| + return |
| + |
| + |
| +def mandatory_parenthesis(x, y, z): |
| + if (): |
| + return |
| + if (x, y, z): |
| + return |
| + |
| + if (x or y) and z: |
| + return |
| + if x and (y or z): |
| + return |
| + |
| + if (x or |
| + y): |
| + return |
« no previous file with comments
|
« flake8-abp/tests/A110.py ('k')
|
flake8-abp/tests/A201.py » ('j')
|
no next file with comments »
