Merge pull request #3076 from dirkmueller/flake8

Use "not in" not "not X in"

.github/workflows/ci-test.yml

@@ -34,9 +34,10 @@ jobs:
         CI_NODE_INDEX: ${{ matrix.ci_node_index }}
 
     - name: Submit coverage report to Codecov
-      uses: codecov/codecov-action@v3
+      uses: codecov/codecov-action@v4
       with:
         fail_ci_if_error: true
+        token: ${{ secrets.CODECOV_TOKEN }}
 
     - name: Stop containers
       if: always()

osclib/repochecks.py

@@ -102,7 +102,7 @@ def _fileconflicts(pfile, arch, target_packages, whitelist):
             sp1 = conflict['between'][0]
             sp2 = conflict['between'][1]
 
-            if not sp1[0] in target_packages and not sp2[0] in target_packages:
+            if sp1[0] not in target_packages and sp2[0] not in target_packages:
                 continue
 
             if _check_conflicts_whitelist(sp1, sp2, whitelist):

osclib/stagingapi.py

@@ -599,7 +599,7 @@ class StagingAPI(object):
 
         ignored_requests = self.get_ignored_requests()
         for rq in root.findall('request'):
-            if not rq.get('id') in ignored_requests:
+            if rq.get('id') not in ignored_requests:
                 requests.append(rq)
         return requests