Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Restore coverage checks of xqueue #887

Closed
timmc-edx opened this issue Apr 13, 2023 · 2 comments
Closed

Restore coverage checks of xqueue #887

timmc-edx opened this issue Apr 13, 2023 · 2 comments

Comments

@timmc-edx
Copy link
Contributor

timmc-edx commented Apr 13, 2023

xqueue has been using the deprecated codecov package, which finally was yanked from PyPI. In order to unblock CI, we've removed the broken coverage step (#886) but this repo should have coverage restored at some point.

Most repos are using the https://github.com/codecov/codecov-action v3 action, but it's not immediately obvious how to get that to work with xqueue, which runs its tests in a docker container. (Do we need to copy a coverage.xml file out of the container? Or perhaps use a different approach entirely for getting MySQL in the testing environment?)

timmc-edx added a commit that referenced this issue Apr 13, 2023
#887 tracks restoring and fixing
coverage testing.
timmc-edx added a commit that referenced this issue Apr 14, 2023
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.
@jristau1984
Copy link

@spencertiberi can you transfer this to the Aurora backlog?

@spencertiberi
Copy link

Done!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants