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

initial push for flixcheck #7481

Open
wants to merge 28 commits into
base: master
Choose a base branch
from
Open

initial push for flixcheck #7481

wants to merge 28 commits into from

Conversation

BongoSort
Copy link

Initial version of QuickCheck for flix, FlixCheck.

This version was used at the meeting 25/03. At this point in time, we do not use monads.

@magnus-madsen magnus-madsen marked this pull request as draft March 27, 2024 09:52
@JonathanStarup
Copy link
Contributor

JonathanStarup commented May 1, 2024

Hey @BongoSort it looks like this PR doesn't have the IO effect around like you showed today, but you can use the regioned random lib here https://github.com/JonathanStarup/random/tree/v1.0.0 if you want. that avoids IO when you have a seed

@BongoSort
Copy link
Author

BongoSort commented May 2, 2024 via email

@BongoSort BongoSort marked this pull request as ready for review May 2, 2024 07:08
/// Represents a splittable pseudorandom number.
pub enum SplittableRandom(##java.util.SplittableRandom)

mod SplittableRandom {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh jeg vidste ikke at i har jeres egen wrapper. Men i kan effect annotere ligesom mit library for at undgå IO når i bruger freshWithSeed

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.

None yet

4 participants