Merge ~cristiangsp/launchpad:linting-f811 into launchpad:master

Proposed by Cristian Gonzalez
Status: Merged
Approved by: Cristian Gonzalez
Approved revision: 40ff7bfe918fd52666f6108740abde6a75054c8a
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cristiangsp/launchpad:linting-f811
Merge into: launchpad:master
Diff against target: 48 lines (+6/-7)
3 files modified
lib/devscripts/tests/test_sourcecode.py (+1/-1)
lib/lp/bugs/scripts/debbugs.py (+0/-2)
lib/lp/services/features/testing.py (+5/-4)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+406647@code.launchpad.net

Commit message

Fixing F811 linting errors

Description of the change

Fixing F811 linting errors

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/lib/devscripts/tests/test_sourcecode.py b/lib/devscripts/tests/test_sourcecode.py
index 6b538c7..ef03676 100644
--- a/lib/devscripts/tests/test_sourcecode.py
+++ b/lib/devscripts/tests/test_sourcecode.py
@@ -112,7 +112,7 @@ class TestInterpretConfiguration(unittest.TestCase):
112 config = interpret_config([['key', 'value;revno=45']], False)112 config = interpret_config([['key', 'value;revno=45']], False)
113 self.assertEqual({'key': ('value', '45', False)}, config)113 self.assertEqual({'key': ('value', '45', False)}, config)
114114
115 def test_key_value_revision(self):115 def test_key_value_revision_with_multiple_revnos_raises_error(self):
116 # A (key, value) pair without a third optional value when the116 # A (key, value) pair without a third optional value when the
117 # value has multiple suffixes of ``;revno=[REVISION]`` raises an117 # value has multiple suffixes of ``;revno=[REVISION]`` raises an
118 # error.118 # error.
diff --git a/lib/lp/bugs/scripts/debbugs.py b/lib/lp/bugs/scripts/debbugs.py
index a15a73b..81055c1 100644
--- a/lib/lp/bugs/scripts/debbugs.py
+++ b/lib/lp/bugs/scripts/debbugs.py
@@ -271,8 +271,6 @@ class Database:
271 return bug271 return bug
272272
273if __name__ == '__main__':273if __name__ == '__main__':
274 import sys
275
276 for bug in Database('/srv/debzilla.no-name-yet.com/debbugs'):274 for bug in Database('/srv/debzilla.no-name-yet.com/debbugs'):
277 try:275 try:
278 print(bug, bug.subject)276 print(bug, bug.subject)
diff --git a/lib/lp/services/features/testing.py b/lib/lp/services/features/testing.py
index e4ed8be..576322f 100644
--- a/lib/lp/services/features/testing.py
+++ b/lib/lp/services/features/testing.py
@@ -67,12 +67,13 @@ class FeatureFixtureMixin:
6767
68 original_controller = get_relevant_feature_controller()68 original_controller = get_relevant_feature_controller()
6969
70 def scope_lookup(scope_name):
71 request = get_current_browser_request()
72 return ScopesFromRequest(request).lookup(scope_name)
73
74 if self.override_scope_lookup:70 if self.override_scope_lookup:
75 scope_lookup = self.override_scope_lookup71 scope_lookup = self.override_scope_lookup
72 else:
73 def scope_lookup(scope_name):
74 request = get_current_browser_request()
75 return ScopesFromRequest(request).lookup(scope_name)
76
76 install_feature_controller(77 install_feature_controller(
77 FeatureController(scope_lookup, rule_source))78 FeatureController(scope_lookup, rule_source))
78 self.addCleanup(install_feature_controller, original_controller)79 self.addCleanup(install_feature_controller, original_controller)

Subscribers

People subscribed via source and target branches

to status/vote changes: