Fix tests for membership to use 'not in'

flake8 reports:

  E713 test for membership should be 'not in'

Change-Id: I4446be67c431b7267105b53478d2ceba2af758d7
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254451
Tested-by: David Pursehouse <dpursehouse@collab.net>
Reviewed-by: Mike Frysinger <vapier@google.com>

2 files changed, 2 insertions, 2 deletions

diff --git a/subcmds/branches.py b/subcmds/branches.py
index fb60d7de..b8958848 100644
--- a/subcmds/branches.py
+++ b/subcmds/branches.py
@@ -158,7 +158,7 @@ is shown, then the branch appears in all projects.
           for b in i.projects:
             have.add(b.project)
           for p in projects:
-            if not p in have:
+            if p not in have:
               paths.append(p.relpath)
 
         s = ' %s %s' % (in_type, ', '.join(paths))
diff --git a/subcmds/sync.py b/subcmds/sync.py
index 13c419c3..54b5a7fc 100644
--- a/subcmds/sync.py
+++ b/subcmds/sync.py
@@ -744,7 +744,7 @@ later is required to fix a server side protocol bug.
     if not opt.quiet:
       print('Using manifest server %s' % manifest_server)
 
-    if not '@' in manifest_server:
+    if '@' not in manifest_server:
       username = None
       password = None
       if opt.manifest_server_username and opt.manifest_server_password: