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

Drop Util.set_ci_printing #180

Merged
merged 6 commits into from
Nov 11, 2022
Merged

Drop Util.set_ci_printing #180

merged 6 commits into from
Nov 11, 2022

Conversation

jmid
Copy link
Collaborator

@jmid jmid commented Nov 11, 2022

This PR eliminates Util.set_ci_printing and thus fixes #144

We achieve the same effect of increasing pause time between printed status lines by utilizing QCHECK_MSG_INTERVAL from the freshly released 0.20 version. For uniformity and simplicity's sake, we therefore require this version as the minimal requirement.

On purpose the PR is built on top of #88 to avoid later rebasing/merging hassle.
The two initial commits can therefore be ignored.

n-osborne and others added 6 commits November 11, 2022 12:36
…d STM_thread

Split STM into sub-libraries STM_base, STM_sequential, STM_domain, and STM_thread
exposed by the qcheck-stm package.
This removes systhreads as a dependency for STM_domain leaving only STM_thread to depend on threads.

Update src/ tests accordingly, splitting src/neg_tests/ref_stm_tests into three files

Co-authored-by: Samuel Hym <[email protected]>
Co-authored-by: Jan Midtgaard <[email protected]>
@jmid jmid mentioned this pull request Nov 11, 2022
@jmid jmid merged commit ec17709 into main Nov 11, 2022
@jmid jmid deleted the elim_util_set_ci_printing branch November 11, 2022 21:49
@jmid jmid mentioned this pull request Nov 15, 2022
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.

Eliminate Util.set_ci_printing
2 participants