pep8: Fix E211 violations.

tools/lint-all

@@ -85,7 +85,7 @@ def check_pep8(files):
         'E402', 'E501', 'W503', 'E711', 'E201', 'E203', 'E202', 'E128', 'E226', 'E124',
         'E126', 'E127', 'E121', 'E122', 'E123', 'E266', 'E265', 'E261', 'E221', 'E303',
         'E241', 'E712', 'E702', 'E401', 'E115', 'E114', 'E111', 'E731', 'E302', 'E129',
-        'E741', 'E714', 'W391', 'E211', 'E713', 'E502', 'E131', 'E305', 'E251', 'E306',
+        'E741', 'E714', 'W391', 'E713', 'E502', 'E131', 'E305', 'E251', 'E306',
     ]
     pep8 = subprocess.Popen(
         ['pycodestyle'] + files + ['--ignore={rules}'.format(rules=','.join(ignored_rules))],

tools/minify-js

@@ -54,7 +54,7 @@ def get_changed_source_files(other_checkout):
         # If git returned an error, assume we can't reuse any files, and
         # regenerate everything.
         print("Warning: git returned an error when comparing to the previous")
-        print ("deploy in %s. Will re-minify JavaScript instead of reusing"
+        print("deploy in %s. Will re-minify JavaScript instead of reusing"
                % (other_checkout,))
         return None
 

zerver/lib/actions.py

@@ -691,7 +691,7 @@ def compute_mit_user_fullname(email):
     except DNS.Base.ServerError:
         pass
     except:
-        print ("Error getting fullname for %s:" % (email,))
+        print("Error getting fullname for %s:" % (email,))
         traceback.print_exc()
     return email.lower()
 

zerver/tests/test_subs.py

@@ -1032,7 +1032,7 @@ class SubscriptionRestApiTest(ZulipTestCase):
         user_profile.full_name = 'Hamlet'
         user_profile.save()
 
-        def method1 (req, user_profile):
+        def method1(req, user_profile):
             # type: (HttpRequest, UserProfile) -> HttpResponse
             user_profile.full_name = 'Should not be committed'
             user_profile.save()