diff options
| author | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 11:24:10 +0900 |
|---|---|---|
| committer | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 05:18:17 +0000 |
| commit | eeff3537de0fe4a6d4f5dc3d04cd2dc28c3a044f (patch) | |
| tree | 27523adfee3445ea85d9dffc622a0eef27e40742 /subcmds/sync.py | |
| parent | 8f78a83083420f0c0b1da64aad3ab1d060fed1a0 (diff) | |
| download | git-repo-eeff3537de0fe4a6d4f5dc3d04cd2dc28c3a044f.tar.gz | |
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>
Diffstat (limited to 'subcmds/sync.py')
| -rw-r--r-- | subcmds/sync.py | 2 |
1 files changed, 1 insertions, 1 deletions
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. | |||
| 744 | if not opt.quiet: | 744 | if not opt.quiet: |
| 745 | print('Using manifest server %s' % manifest_server) | 745 | print('Using manifest server %s' % manifest_server) |
| 746 | 746 | ||
| 747 | if not '@' in manifest_server: | 747 | if '@' not in manifest_server: |
| 748 | username = None | 748 | username = None |
| 749 | password = None | 749 | password = None |
| 750 | if opt.manifest_server_username and opt.manifest_server_password: | 750 | if opt.manifest_server_username and opt.manifest_server_password: |
