-
Notifications
You must be signed in to change notification settings - Fork 8
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
Run everything on 1.10 #385
Conversation
Codecov ReportAttention: Patch coverage is
... and 8 files with indirect coverage changes 🚨 Try these New Features:
|
Performance Ratio:
|
This is getting there, but there are a few issues on 1.10 that need dealing with. I've made some progress today, and hope to finish off tomorrow. |
@lcontento it looks like we're nearly there -- down to one error. I have to call it a day now -- should be able to resolve in the morning! |
#384 points out that the workaround I introduced in #377 only works on 1.11. I didn't catch this because, so far, I've only been running a subset of CI on 1.11 in order to keep CI times down. This PR will
If it looks like the amount of time spent running CI is becoming prohibitive again, I'll have to look at ways to speed it up.