diff --git a/.github/workflows/ci-test.yml b/.github/workflows/ci-test.yml index 87d4d116..8c8e875f 100644 --- a/.github/workflows/ci-test.yml +++ b/.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() diff --git a/osclib/repochecks.py b/osclib/repochecks.py index 368b79ad..73f6b8b4 100644 --- a/osclib/repochecks.py +++ b/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): diff --git a/osclib/stagingapi.py b/osclib/stagingapi.py index a9b8d36d..1b5e8609 100644 --- a/osclib/stagingapi.py +++ b/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