From eeff3537de0fe4a6d4f5dc3d04cd2dc28c3a044f Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Wed, 12 Feb 2020 11:24:10 +0900 Subject: 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 Reviewed-by: Mike Frysinger --- subcmds/sync.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'subcmds/sync.py') 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: -- cgit v1.2.3-54-g00ecf