From 8f78a83083420f0c0b1da64aad3ab1d060fed1a0 Mon Sep 17 00:00:00 2001 From: David Pursehouse Date: Wed, 12 Feb 2020 11:20:36 +0900 Subject: 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 Reviewed-by: Mike Frysinger --- subcmds/upload.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'subcmds/upload.py') 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/ key = 'review.%s.autoreviewer' % project.GetBranch(name).remote.review raw_list = project.config.GetString(key) - if not raw_list is None: + if raw_list is not None: people[0].extend([entry.strip() for entry in raw_list.split(',')]) key = 'review.%s.autocopy' % project.GetBranch(name).remote.review raw_list = project.config.GetString(key) - if not raw_list is None and len(people[0]) > 0: + if raw_list is not None and len(people[0]) > 0: people[1].extend([entry.strip() for entry in raw_list.split(',')]) def _FindGerritChange(self, branch): -- cgit v1.2.3-54-g00ecf