[Git][ftp-team/dak][master] 2 commits: Fix E713: Test for membership should be 'not in'

[Joerg Jaspert <joerg@debian.org>]

Title: GitLab

Joerg Jaspert pushed to branch master at Debian FTP Team / dak

Commits:

• 14a31f10
  by Bastian Blank at 2018-05-19T20:45:51+00:00

  Fix E713: Test for membership should be 'not in'
  

• e663bccb
  by Joerg Jaspert at 2018-05-19T20:46:08+00:00

  Merge branch 'fix-E713' into 'master'
  
  Fix E713: Test for membership should be 'not in'
  
  See merge request ftp-team/dak!53

5 changed files:

• dak/bts_categorize.py
• dak/cruft_report.py
• dak/process_new.py
• setup.cfg
• tests/db_test.py

Changes:

dak/bts_categorize.py

---

...	...	@@ -100,7 +100,7 @@ class BugClassifier(object):
100	100	tagged_bugs_ftp += tagged_bugs[tags]
101	101
102	102	return [ bug for bug in bts.get_status( bts.get_bugs("package", "ftp.debian.org" ) ) \
103		- if bug.pending=='pending' and not bug.bug_num in tagged_bugs_ftp ]
	103	+ if bug.pending=='pending' and bug.bug_num not in tagged_bugs_ftp ]
104	104
105	105	def classify_bug(self, bug):
106	106	"""

dak/cruft_report.py

---

...	...	@@ -485,9 +485,9 @@ def report_outdated_nonfree(suite, session, rdeps=False):
485	485	arch = package[2]
486	486	if arch == 'all':
487	487	continue
488		- if not source in packages:
	488	+ if source not in packages:
489	489	packages[source] = {}
490		- if not binary in packages[source]:
	490	+ if binary not in packages[source]:
491	491	packages[source][binary] = set()
492	492	packages[source][binary].add(arch)
493	493	if packages:

dak/process_new.py

---

...	...	@@ -762,7 +762,7 @@ def sort_uploads(new_queue, uploads, session, nobinaries=False):
762	762	filter_by(trainee=False, policy_queue=new_queue).distinct()]
763	763	for upload in uploads:
764	764	source = upload.changes.source
765		- if not source in sources:
	765	+ if source not in sources:
766	766	sources[source] = []
767	767	sources[source].append({'upload': upload,
768	768	'date': upload.changes.created,

setup.cfg

---

...	...	@@ -36,7 +36,6 @@ ignore =
36	36	E502,
37	37	E703,
38	38	E704,
39		- E713,
40	39	E722,
41	40	E731,
42	41	E741,

tests/db_test.py

---

...	...	@@ -140,7 +140,7 @@ class DBDakTestCase(DakTestCase):
140	140	for f in self.file.values():
141	141	f.sha1sum = 'sha1sum'
142	142	f.sha256sum = 'sha256sum'
143		- if not 'gnome-hello_3.0-1' in f.filename:
	143	+ if 'gnome-hello_3.0-1' not in f.filename:
144	144	archive_files.append(ArchiveFile(
145	145	archive=self.archive, component=self.comp['main'], file=f))
146	146	else:

—
View it on GitLab.
You're receiving this email because of your account on salsa.debian.org. If you'd like to receive fewer emails, you can adjust your notification settings.