openedx
Remove Paver code specific to testing
As we have stopped using paver for testing on our CI and also want to remove paver altogether in the future, so we have to remove the codebase specific for testing in the paver. Find all those files (if multiple) specific for testing and remove those, and also delete any reference to those. We also need to update our testing documentation in the edx-platform. For that, we will need subsequent PR in devstack too also to update documentation there.
A/C
- Drop all codebase specific to testing in edx-platform
- Update testing documentation to use
pytestdirectly
Do we still provide support for Circle CI tests? that script is still using paver for testing. I think we can drop support for that because we have already moved to github actions and also made separate Github actions for forks
cc @kdmccormick @jmbowman
I think that should be fine; the Circle CI file hasn't been updated in 4.5 years, and as you say we now have a replacement that works out of the box in any fork. And we have precedent for this, even a runbook: https://openedx.atlassian.net/wiki/spaces/AC/pages/3372974153/How+to+remove+CircleCI+checks+from+an+openedx+repo . Might be worth a quick DEPR ticket that goes through the full process quickly, mostly as an inform. It looks like this is the last circle.yml file anywhere in the edx or openedx GitHub orgs.
