Skip to content
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

Require qcheck-core.0.23 #491

Merged
merged 6 commits into from
Dec 13, 2024
Merged

Require qcheck-core.0.23 #491

merged 6 commits into from
Dec 13, 2024

Conversation

jmid
Copy link
Collaborator

@jmid jmid commented Dec 13, 2024

This PR bumps the qcheck-core lower bound to 0.23.

This freshly-released version means the Util.Pp.pp_fun_ output will contain properly escaped and quoted chars and strings, and thus more comprehensible - see, e.g., the expect test diff in ef6c006.

As an added benefit, the new lower-bound means the Lin and STM show functions can be simplified,
as they no longer need to work around QCheck.Print's lack of quoting and escaping.

@jmid jmid merged commit 1965542 into main Dec 13, 2024
31 checks passed
@jmid jmid deleted the require-qcheck-0-23 branch December 13, 2024 22:40
@jmid
Copy link
Collaborator Author

jmid commented Dec 14, 2024

CI summary for 2684c1b: all 31 workflows pass.

CI summary for merge to main: all 32 workflows pass.

@jmid jmid mentioned this pull request Dec 14, 2024
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.

1 participant