Add ArbitraryDequeue to scalacheck module and wire ArbitraryTreeList#801
Add ArbitraryDequeue to scalacheck module and wire ArbitraryTreeList#801krrishverma1805-web wants to merge 1 commit intotypelevel:masterfrom
Conversation
|
Hi @gemelen 👋 Would you have a chance to take a look at this when you get a moment? This PR addresses one of the items in TODO.md — moving arbitrary Happy to make any adjustments based on your feedback. Thanks! |
|
Hi @krrishverma1805-web, before we look at your PR, would you mind making a contribution to our onboarding repository? Directions are in the README. |
|
Hi @armanbilge , thanks for the pointer! I'll complete the onboarding and come back. 🙂 |
|
I've completed the onboarding contribution: |
|
Hi @gemelen, the onboarding PR has been merged! Looking forward to any feedback on this one. |
|
@krrishverma1805-web
|
What this PR does
ArbitraryDequeueinstance to thecats-collections-scalacheckmodule, makingArbitrary[Dequeue[A]]available to downstream users.ArbitraryTreeListintoAllArbitraryand the package object, which was previously defined but not included.Why
The
TODO.mdnotes: "move arbitrary instances somewhere that one can import into other projects".Currently,
Arbitrary[Dequeue[A]]is only defined inline inDequeueSuiteand is not available to downstream library users who may want to write property-based tests involvingDequeue. This PR makes it available through the standard import paths: