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

Delta Between Two Patch Sets: flake8-abp/tests/A112.py

Issue 29342824: Issue 4044 - Added handling for __future__ unicode_literals import to check_quotes() (Closed)
Left Patch Set: Updates readme for clarity, moves unicode prefix check out of single line and docstring if statemen… Created May 26, 2016, 9:17 p.m.
Right Patch Set: removed redundant comment Created June 2, 2016, 5:45 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « flake8-abp/tests/A110_unicode_literals.py ('k') | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 class Foo: 1 # A112
2 def unicode(self): 2 u"""This is a bad docstring"""
3 # A112
4 u"""This should not be a unicode docstring"""
Sebastian Noack 2016/05/27 12:33:48 This test code can be simplified by using a functi
5 3
6 4 # * A112
7 def foo(): 5 foo = u'nope'
8 return [
Sebastian Noack 2016/05/27 12:33:48 This test code can b e simplified by just returnin
9 # A112
10 u'yeahyeah'
11 ]
LEFTRIGHT

Powered by Google App Engine
This is Rietveld