-
Notifications
You must be signed in to change notification settings - Fork 8
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
Split (or merge) the library? #69
Comments
This has been discussed over the years in hedgehogqa/fsharp-hedgehog#182. One big change since the last comment on that issue is that I am now the lead maintainer of both projects. My preference is to merge F# Hedgehog Experimental into F# Hedgehog. Would you like to contribute a PR for that? |
Maybe. I will see if I can make time for it, but I cannot promise. |
👍 I believe it's a good choice - even though my opinion is still the same as in 2019 - it'll make Hedgehog.Xunit depend only on Hedgehog, for example. And Hedgehog.Experimental users can just add/install Hedgehog. Perhaps one thing I'd consider is making this move explicit, e.g. either keeping everything under GenX, or moving everything under Gen but under its own namespace. Of course, @TysonMN will have the final word. |
Hmm... If the merge happens, fsharp-hedgehog will take an extra dependency, TypeShape. Just one more thing to consider. |
Possibly relevant: ploeh/ploeh.github.com#855 (comment) Regarding TypeShape: It is used for the auto-generator. It is likely possible to write the generator without TypeShape. Not sure how much work that'd be, though. In any case, since Hedgehog is used for test projects, not production projects, I'm personally not concerned about the dependency. |
It looks like the library consists of two parts:
Hedgehog
. These are all thesesorted
,noNull
,shuffle
,uri
generators and operations on themAutoGenConfig
,GenX.auto
, etc.)These parts do not seem to depend on each other.
The questions are:
Gen
/Range
combinators and constructors? Some of them may even go to HH itself.Hedgehog.Autogen
can be born as its own library?The text was updated successfully, but these errors were encountered: