Skip to content

Conversation

@jmid
Copy link
Collaborator

@jmid jmid commented Dec 21, 2025

The QCheck.0.90 release revises the API, deprecating a number of previous bindings:
https://github.com/c-cube/qcheck/releases/tag/v0.90

With the help of https://github.com/v-gb/ocamlmig and the forthcoming annotations from c-cube/qcheck#398, I've been able to migrate to the new API without much effort.

In a subsequent pass, I then reviewed the changes:

  • Here I discovered Lin.int_small would be translated to use nat_small (for a distribution-preserving mapping) which excludes negative numbers, and therefore changed it to instead use the new int_small including negative numbers. In effect this creates an API change of behavior.
  • I also simplified some array combinators to use array_small.

Let's see if the changes causes the CI test suite to behave any different...

@jmid
Copy link
Collaborator Author

jmid commented Dec 22, 2025

CI summary:

Out of 51 workflows 2 failed with genuine errors

@jmid jmid merged commit af02483 into main Dec 22, 2025
65 of 69 checks passed
@jmid jmid deleted the qcheck.0.90-migration branch December 22, 2025 22:27
@jmid
Copy link
Collaborator Author

jmid commented Dec 23, 2025

CI summary for merge to main:

Out of 52 workflows 1 failed with a genuine error

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants