Merge lp:~gz/brz/lint_E999 into lp:brz

Proposed by Martin Packman
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~gz/brz/lint_E999
Merge into: lp:brz
Diff against target: 99 lines (+14/-10)
5 files modified
breezy/git/tests/test_workingtree.py (+2/-3)
breezy/tests/per_workingtree/test_workingtree.py (+2/-2)
setup.cfg (+2/-1)
tools/capture_tree.py (+4/-2)
tools/prepare_for_latex.py (+4/-2)
To merge this branch: bzr merge lp:~gz/brz/lint_E999
Reviewer Review Type Date Requested Status
Jelmer Vernooij Approve
Review via email: mp+358945@code.launchpad.net

Commit message

Fix E999 lint error for Python 2 flake8.

Description of the change

Fix E999 lint error for Python 2 flake8

Ignore D and I flake8 plugin rule prefixes for now.

Fix some newly introduced rule breakage.

To post a comment you must log in.
Revision history for this message
Jelmer Vernooij (jelmer) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/git/tests/test_workingtree.py'
--- breezy/git/tests/test_workingtree.py 2018-11-16 22:18:20 +0000
+++ breezy/git/tests/test_workingtree.py 2018-11-17 16:31:16 +0000
@@ -249,10 +249,9 @@
249 a = Blob.from_string(b'irrelevant\n')249 a = Blob.from_string(b'irrelevant\n')
250 with self.wt.lock_tree_write():250 with self.wt.lock_tree_write():
251 (index, index_path) = self.wt._lookup_index(b'a')251 (index, index_path) = self.wt._lookup_index(b'a')
252 index[b'a'] = IndexEntry(252 index[b'a'] = IndexEntry(0, 0, 0, 0, S_IFGITLINK, 0, 0, 0, a.id, 0)
253 0, 0, 0, 0, S_IFGITLINK, 0, 0, 0, a.id, 0)
254 self.wt._index_dirty = True253 self.wt._index_dirty = True
255 t = Tree()254 t = Tree()
256 t.add(b"a", S_IFGITLINK , a.id)255 t.add(b"a", S_IFGITLINK, a.id)
257 self.store.add_object(t)256 self.store.add_object(t)
258 self.expectDelta([], tree_id=t.id)257 self.expectDelta([], tree_id=t.id)
259258
=== modified file 'breezy/tests/per_workingtree/test_workingtree.py'
--- breezy/tests/per_workingtree/test_workingtree.py 2018-11-16 23:21:31 +0000
+++ breezy/tests/per_workingtree/test_workingtree.py 2018-11-17 16:31:16 +0000
@@ -1077,8 +1077,8 @@
1077 os.unlink('tree/a')1077 os.unlink('tree/a')
1078 try:1078 try:
1079 self.assertEqual(1079 self.assertEqual(
1080 {'a', 'b', ''},1080 {'a', 'b', ''},
1081 set(tree.all_versioned_paths()))1081 set(tree.all_versioned_paths()))
1082 except errors.UnsupportedOperation:1082 except errors.UnsupportedOperation:
1083 raise TestNotApplicable('tree does not support all_file_ids')1083 raise TestNotApplicable('tree does not support all_file_ids')
10841084
10851085
=== modified file 'setup.cfg'
--- setup.cfg 2018-11-16 19:38:57 +0000
+++ setup.cfg 2018-11-17 16:31:16 +0000
@@ -3,6 +3,8 @@
3# because even with the lazy import plugin it still triggers3# because even with the lazy import plugin it still triggers
4# for lazy_import statements before other imports.4# for lazy_import statements before other imports.
5ignore =5ignore =
6 D
7 I
6 E1238 E123
7 E2419 E241
8 E25110 E251
@@ -29,7 +31,6 @@
29 E72231 E722
30 E73132 E731
31 E74133 E741
32 E999
33 F40134 F401
34 F40235 F402
35 F40336 F403
3637
=== modified file 'tools/capture_tree.py'
--- tools/capture_tree.py 2017-05-22 00:56:52 +0000
+++ tools/capture_tree.py 2018-11-17 16:31:16 +0000
@@ -2,14 +2,16 @@
22
3# Copyright (C) 2005 Canonical Ltd3# Copyright (C) 2005 Canonical Ltd
44
5"""Print to stdout a description of the current directory, 5"""Print to stdout a description of the current directory,
6formatted as a Python data structure.6formatted as a Python data structure.
77
8This can be useful in tests that need to recreate directory8This can be useful in tests that need to recreate directory
9contents."""9contents."""
1010
11from __future__ import print_function
12
13import os
11import sys14import sys
12import os
1315
14from breezy.trace import enable_default_logging16from breezy.trace import enable_default_logging
15enable_default_logging()17enable_default_logging()
1618
=== modified file 'tools/prepare_for_latex.py'
--- tools/prepare_for_latex.py 2018-11-16 13:02:50 +0000
+++ tools/prepare_for_latex.py 2018-11-17 16:31:16 +0000
@@ -26,6 +26,8 @@
26# along with this program; if not, write to the Free Software26# along with this program; if not, write to the Free Software
27# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA27# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
2828
29from __future__ import print_function
30
29import os31import os
30import re32import re
31import shutil33import shutil
@@ -35,8 +37,8 @@
3537
36verbose = False38verbose = False
3739
38IMAGE_DIRECTIVE_PATTERN = re.compile(ur'^..\s+image::\s+(.*)\s+$')40IMAGE_DIRECTIVE_PATTERN = re.compile(u'^..\\s+image::\\s+(.*)\`\s+$')
39DIRECTIVE_ELEMENT_PATTERN = re.compile(ur'^\s+:[^:]+:\s+')41DIRECTIVE_ELEMENT_PATTERN = re.compile(u'^\\s+:[^:]+:\\s+')
4042
41class Converter(object):43class Converter(object):
42 def __init__(self, srcdir, destdir):44 def __init__(self, srcdir, destdir):

Subscribers

People subscribed via source and target branches