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 | |
| 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>
| -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): |
