Delta Between Two Patch Sets: flake8-abp/tests/A201.py
Issue 29340727:
Noissue - Added flake8 extension accounting for our coding style and some other stuff (Closed)
Left Patch Set: Added check for redundant paranthesis
Created April 22, 2016, 2:22 p.m.
Right Patch Set: Addressed comments, fixed two bugs, use ascii()
Created May 9, 2016, 4:47 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« flake8-abp/tests/A111.py ('k')
|
flake8-abp/tests/A202.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 # A201 | |
| 2 global a | |
| 3 a = 1 | |
| 4 | |
| 5 | |
| 6 class Foo: | |
| 7 global a | |
| 8 # A201 | |
| 9 global a, b | |
| 10 a = 2 | |
| 11 b = 3 | |
| 12 | |
| 13 def foo(self): | |
| 14 # A201 | |
| 15 global b, c | |
| 16 b = 4 | |
| 17 | |
| 18 | |
| 19 math = None | |
| 20 Class = None | |
| 21 func = None | |
| 22 | |
| 23 | |
| 24 def lazy_import(): | |
| 25 global math | |
| 26 if math is None: | |
| 27 import math | |
| 28 return math | |
| 29 | |
| 30 | |
| 31 def lacy_class(): | |
| 32 global Class | |
| 33 if Class is None: | |
| 34 class Class: | |
| 35 pass | |
| 36 return Class | |
| 37 | |
| 38 | |
| 39 def lazy_function(): | |
| 40 global func | |
| 41 if func is None: | |
| 42 def func(): | |
| 43 pass | |
| 44 return func | |
| LEFT | RIGHT |
« flake8-abp/tests/A111.py ('k')
|
flake8-abp/tests/A202.py » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
