run latest version of openmm in CI #345
Conversation
mikemhenry
changed the title
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #345 +/- ##
=======================================
Coverage 69.57% 69.57%
=======================================
Files 5 5
Lines 825 825
=======================================
Hits 574 574
Misses 251 251 ☔ View full report in Codecov by Sentry. |
| - pytest-xdist | ||
| - pytest-randomly | ||
| - openmm <8.1.1 | ||
| - openmm |
There was a problem hiding this comment.
In principle you'd want to run against 8.1.2 and something older, but I don't care too much about that
There was a problem hiding this comment.
That is a good idea to catch regressions, I'll make an issue and get to it someday
There was a problem hiding this comment.
@mattwthompson decided it was easy enough to add to this PR
|
Looks good! I just need to clean up the branch rules so it's not waiting for tests that never come. |
No description provided.