-
Notifications
You must be signed in to change notification settings - Fork 652
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
[CI] Add post deployment conda and pypi cron tests #4210
Conversation
Linter Bot Results:Hi @IAlibay! Thanks for making this PR. We linted your code and found the following: There are currently no issues detected! 🎉 |
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## develop #4210 +/- ##
========================================
Coverage 93.62% 93.62%
========================================
Files 193 193
Lines 25295 25295
Branches 4063 4063
========================================
Hits 23683 23683
Misses 1096 1096
Partials 516 516 ☔ View full report in Codecov by Sentry. |
Ok current CI test failuires are down to known issues so I won't fix them here:
@MDAnalysis/coredevs if I can get a review / merge here we can move forward towards fixing #4211 |
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.
This looks good as far as I can tell. I don’t know how you’ve tested that the cron job ran but I trust your report.
Thank you for adding these tests!
Thanks! Cron jobs were just enabled to run on this PR until ~ 2 commits ago. |
Please merge at your leisure. |
Fixes #4208
Changes made in this Pull Request:
PR Checklist
📚 Documentation preview 📚: https://mdanalysis--4210.org.readthedocs.build/en/4210/