-
Notifications
You must be signed in to change notification settings - Fork 903
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
Migrate Python tests to GitHub actions #4315
Migrate Python tests to GitHub actions #4315
Conversation
Any thoughts here @FelixDQ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cool! Should this PR also remove the .circleci job then?
It could but the new CI jobs aren't marked as required yet so I have held off on doing that for now. I think it's best to merge these, let them run for a little while to make sure it's consistent then mark them as required and remove the circleCI jobs once we have the same functionality. (the last job to migrate is the web build job) |
@FelixDQ made the changes you requested, there should be nothing blocking this now. 🙂 |
Most of this PR is a direct port of the CircleCI job but I have added review comments for the changes that are not straight forward so please have a look at those.