-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Law testing documentation improvement #3068
Comments
Sorry about the inconveniences, it's an oversight on our part, I did update the doc to point to the new way to use Catsuite, but we didn't publish the website, which I will do shortly. In the meantime, I think it's a good idea to mention dependency needed if you are not using Catsuite: a milestone release of discipline-scalatest in addition to cats-laws. A PR will be great! |
miklos-martin
added a commit
to miklos-martin/cats
that referenced
this issue
Sep 19, 2019
LukaJCB
pushed a commit
that referenced
this issue
Sep 19, 2019
Fixed in #3072, thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi,
First of all, thank you all for your amazing work on cats.
I have recently upgraded to 2.0 and I had a semi-hard time figuring out how to make my tests compile again. Previously I have been using
CatsSuite
fromcats-testkit
. If I understand correctly, my tests broke because of 9ef3af0.I had some trouble following the Laws Testing docs and I think it would be nice to mention there what additional dependencies one will need if they are using scalatest, or specs2 (I'm using scalatest and I needed to add a snapshot version of it to have
org.scalatest.funsuite.AnyFunSuite
and a milestone release ofdiscipline-scalatest
in addition tocats-laws
to haveorg.typelevel.discipline.scalatest.Discipline
).I could send a PR if you like, what do you think?
The text was updated successfully, but these errors were encountered: