search

cdepillabout / password

datatypes and functions for easily working with passwords in Haskell
http://hackage.haskell.org/package/password
55 stars 16 forks source link

Dropping `cabal-doctest` #52

Open tchoutri opened 3 years ago

tchoutri commented 3 years ago

Hi @cdepillabout ! I would like to start the discussion about dropping cabal-doctest. It is deprecated, was never intended to be a sustainable solution, and poses a risk for downstream projects who depend (transitively or not) on password.

cdepillabout commented 3 years ago

The problem is that cabal-doctest was the only reliable solution for quite a while!

Now I think cabal-docspec is available, but I haven't actually tried it with any of my repos. Although I'd be happy to switch to cabal-docspec (or really anything else) if someone is willing to put in the time for figuring out how to migrate.

There's a similar issue for another one of my projects: https://github.com/cdepillabout/pretty-simple/issues/82

tchoutri commented 3 years ago

I hear you, I'm not criticising the original decision, our doctest tooling is pretty appalling. The problem with cabal-docspec is that it suffers from the same lack of maintenance will, so it may become a liability. The upside is that it doesn't encroach in the code that is sent to the user, as you can simply get the tarball during CI from GitHub and run it. :)

tchoutri commented 3 years ago

@cdepillabout I've opened https://github.com/cdepillabout/password/pull/53 :)

cdepillabout commented 3 years ago

@tchoutri I was wondering what your personal motivation is for this. Are you trying to cross-compile password?

tchoutri commented 3 years ago

@cdepillabout Nothing of the sort, I just had a chat with phadej about Cabal 3.6 support and he told me what I say here: He's not proving any maintenance anymore (although someone stepped in but for how long?) and that it was never meant to be a viable solution on the long run.