diff options
author | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 11:20:36 +0900 |
---|---|---|
committer | David Pursehouse <dpursehouse@collab.net> | 2020-02-12 05:17:49 +0000 |
commit | 8f78a83083420f0c0b1da64aad3ab1d060fed1a0 (patch) | |
tree | ea7195188b45ab022671a9f660fc5854fbf12586 /subcmds/upload.py | |
parent | e5913ae41058ddd7f692e9cc18e5f71b8251bd60 (diff) | |
download | git-repo-8f78a83083420f0c0b1da64aad3ab1d060fed1a0.tar.gz |
upload: Fix tests for object identity to use 'is not'
flake8 reports:
E714 test for object identity should be 'is not'
Change-Id: Ib8c4100babaf952bbfe65fd56555ece8a958e4b0
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/254450
Tested-by: David Pursehouse <dpursehouse@collab.net>
Reviewed-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'subcmds/upload.py')
-rw-r--r-- | subcmds/upload.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/upload.py b/subcmds/upload.py index b81cf0a8..f81491a2 100644 --- a/subcmds/upload.py +++ b/subcmds/upload.py | |||
@@ -326,12 +326,12 @@ Gerrit Code Review: https://www.gerritcodereview.com/ | |||
326 | 326 | ||
327 | key = 'review.%s.autoreviewer' % project.GetBranch(name).remote.review | 327 | key = 'review.%s.autoreviewer' % project.GetBranch(name).remote.review |
328 | raw_list = project.config.GetString(key) | 328 | raw_list = project.config.GetString(key) |
329 | if not raw_list is None: | 329 | if raw_list is not None: |
330 | people[0].extend([entry.strip() for entry in raw_list.split(',')]) | 330 | people[0].extend([entry.strip() for entry in raw_list.split(',')]) |
331 | 331 | ||
332 | key = 'review.%s.autocopy' % project.GetBranch(name).remote.review | 332 | key = 'review.%s.autocopy' % project.GetBranch(name).remote.review |
333 | raw_list = project.config.GetString(key) | 333 | raw_list = project.config.GetString(key) |
334 | if not raw_list is None and len(people[0]) > 0: | 334 | if raw_list is not None and len(people[0]) > 0: |
335 | people[1].extend([entry.strip() for entry in raw_list.split(',')]) | 335 | people[1].extend([entry.strip() for entry in raw_list.split(',')]) |
336 | 336 | ||
337 | def _FindGerritChange(self, branch): | 337 | def _FindGerritChange(self, branch): |