Skip to content

Commit

Permalink
build: Remove yanked codecov package (and coverage upload) (#886)
Browse files Browse the repository at this point in the history
Stop trying to install the deprecated (and now yanked) codecov package.

Also remove the CI step that was trying to use it, since we'd prefer to unblock CI now
and restore coverage later. #887
tracks restoring coverage testing.
  • Loading branch information
timmc-edx authored Apr 14, 2023
1 parent b8ef63b commit 592888d
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 33 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,9 +29,3 @@ jobs:
run: |
docker exec xqueue bash -c "cd /edx/app/xqueue/xqueue/; pip3 install -r requirements/ci.txt"
docker exec xqueue bash -c "cd /edx/app/xqueue/xqueue/ && DB_HOST=${{ matrix.db-version }} tox -e ${TOXENV}"
- name: Code Coverage
if: matrix.tox-env=='django32'
run: |
python3.8 -m pip install -r requirements/ci.txt &&
docker exec xqueue bash -c "cd /edx/app/xqueue/xqueue/; coverage xml" && codecov
1 change: 0 additions & 1 deletion requirements/ci.in
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
# Requirements for running tests in Github CI
-c constraints.txt

codecov # Code coverage reporting
tox # Virtualenv management for tests
tox-battery # Makes tox aware of requirements file changes
14 changes: 0 additions & 14 deletions requirements/ci.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,22 +4,12 @@
#
# make upgrade
#
certifi==2022.12.7
# via requests
charset-normalizer==3.1.0
# via requests
codecov==2.1.12
# via -r requirements/ci.in
coverage==7.2.1
# via codecov
distlib==0.3.6
# via virtualenv
filelock==3.9.1
# via
# tox
# virtualenv
idna==3.4
# via requests
packaging==23.0
# via tox
platformdirs==3.1.1
Expand All @@ -28,8 +18,6 @@ pluggy==1.0.0
# via tox
py==1.11.0
# via tox
requests==2.28.2
# via codecov
six==1.16.0
# via tox
tomli==2.0.1
Expand All @@ -41,7 +29,5 @@ tox==3.28.0
# tox-battery
tox-battery==0.6.1
# via -r requirements/ci.in
urllib3==1.26.15
# via requests
virtualenv==20.21.0
# via tox
13 changes: 1 addition & 12 deletions requirements/dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ build==0.10.0
# pip-tools
certifi==2022.12.7
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# requests
cffi==1.15.1
Expand All @@ -36,7 +35,6 @@ cffi==1.15.1
# pynacl
charset-normalizer==3.1.0
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# requests
click==8.1.3
Expand All @@ -45,13 +43,9 @@ click==8.1.3
# -r requirements/test.txt
# edx-django-utils
# pip-tools
codecov==2.1.12
# via -r requirements/ci.txt
coverage[toml]==7.2.1
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# codecov
# pytest-cov
distlib==0.3.6
# via
Expand Down Expand Up @@ -92,7 +86,6 @@ gunicorn==20.1.0
# via -r requirements/test.txt
idna==3.4
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# requests
iniconfig==2.0.0
Expand Down Expand Up @@ -188,10 +181,7 @@ pyyaml==6.0
# -r requirements/test.txt
# edx-django-release-util
requests==2.28.2
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# codecov
# via -r requirements/test.txt
s3transfer==0.6.0
# via
# -r requirements/test.txt
Expand Down Expand Up @@ -231,7 +221,6 @@ tox-battery==0.6.1
# via -r requirements/ci.txt
urllib3==1.26.15
# via
# -r requirements/ci.txt
# -r requirements/test.txt
# botocore
# requests
Expand Down

0 comments on commit 592888d

Please sign in to comment.