search

TheSeamau5 / elm-check

Property Based Testing in Elm
70 stars 20 forks source link

Remove elm-html dependency #26

Open rtfeldman opened 9 years ago

rtfeldman commented 9 years ago

We started using elm-html 4.0.0 for our prod code, but now our tests break because elm-check requires 3.0.0 - even though we aren't using elm-check's HTML reporting!

It seems like the HTML reporting should be a separate, decoupled package to prevent this. Otherwise it will be either a constant need to upgrade immediately to the latest elm-html (even when there's no need) or else compatibility will continue to break for users who aren't even using the html output.

Thoughts?

TheSeamau5 commented 9 years ago

I think this makes sense. I hadn't thought of it like this before