Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Delta Between Two Patch Sets: flake8-abp/tests/A207.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.
Jump to:
Right: Side by side diff | Download
« no previous file with change/comment | « flake8-abp/tests/A206.py ('k') | flake8-abp/tests/A301.py » ('j') | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
(no file at all)
1 def make_dict():
2 return {
3 'foo': None,
4 'bar': None,
5 0: None,
6 1: None,
7
8 # A207
9 'foo': None,
10 # A207
11 ['bar' + ' '][0].strip(): None,
12 # A207
13 0.0: None,
14 # A207
15 True: None,
16 }
17
18
19 def make_set():
20 return {
21 'foo',
22 'bar',
23 0,
24 1,
25
26 # A207
27 'foo',
28 # A207
29 ['bar' + ' '][0].strip(),
30 # A207
31 0.0,
32 # A207
33 True,
34 }
LEFTRIGHT

Powered by Google App Engine
This is Rietveld