-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Align QCheck 0.9 and 0.10 versions #24703
Conversation
This MR is a strict improvement ("consider for merge"). But there are some interesting errors in the CI:
We could use the opportunity to fix those, or we can do it in a follow-up MR. |
I agree Raphael! That one happens when testing |
The latest commit gets the 0.9-0.17 packages linting cleanly |
From
I guess this can be solved by either
I've gone for the latter in d545271. |
#24734 fixes missing lower bounds for some of revdeps. |
137b25f
to
ac2e54a
Compare
Rebased on |
Finally, I'm starting to suspect that this is a coincidence: If so, extending the lower bound search to test dependencies would be a nice feature to add... 🙏 🙂 |
Thanks |
The
qcheck*
opam files allow mixing versions across releases such aswhich ends up triggering a fadbadml lower bound failure in #24702:
For all
qcheck*
releases from 0.11 and above their versions are aligned.This PR extends that range to uniformly include 0.9 and 0.10 too.