-
-
Notifications
You must be signed in to change notification settings - Fork 141
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
Fix the Github Action CI for Python tests #445
Fix the Github Action CI for Python tests #445
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #445 +/- ##
=======================================
Coverage ? 98.54%
=======================================
Files ? 19
Lines ? 3360
Branches ? 547
=======================================
Hits ? 3311
Misses ? 26
Partials ? 23 ☔ View full report in Codecov by Sentry. |
Ok, took a few adjustments but all tests are passing now! Sorry for all the pushes, I still am not aware of a way to test GH Actions locally... I am pushing just to see if something works 😕 If anyone has advice on how to test locally and avoid this approach, please let me know! Updated list of changesI had to bump a few action versions up where the older ones caused an issue, and updated the docs requirements installation to take advantage of the modern python packaging layout that happened recently in #406.
Any thoughts on those last 2 hanging points? |
Alright that should be good @raphaelvallat! I also removed the platform and python version from the docs artifact. I added it before, but it's only necessary if you're uploading artifacts from multiple platforms/versions. That's not the case here, so it's cleaner as it was without. I don't see the same web interface for artifacts as you do in the screenshot, but I found the URL in the logs as you suggested (thanks for the pics, very helpful). Agreed, not a big deal for now. |
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.
Amazing thank you! Merging now
* CI95% --> CI95 * p-* --> p_* (also U-* and W-*) * CI[97.5%] --> CI97.5 * mean(A) --> mean_A (also std) * T-test --> T_test (index) * effect sizes * updated notebooks * Fix the Github Action CI for Python tests (#445) * GH Action on main branch instead of master and develop * bump actions/upload-artifact@v2 to v4 * install doc requirements from .toml * bump actions/checkout@v2 --> v4 * bump actions/setup-python@v1 --> v5 * move pip-install-docs back to only run during docs build * typo 3.8 --> 3.9 * bump codecov/codecov-action@v1 --> v4 * remove platform specification from docs-artifact * CI95% --> CI95 * p-* --> p_* (also U-* and W-*) * CI[97.5%] --> CI97.5 * mean(A) --> mean_A (also std) * T-test --> T_test (index) * effect sizes * updated notebooks
This PR corrects a typo in the Python tests Github Action CI yaml file that was preventing the tests from running on PRs. The pingouin master branch was recently renamed from
master
tomain
, and now the yaml file reflects that. This was identified in a comment in #443.