search

typelevel / algebra

Experimental project to lay out basic algebra type classes
https://typelevel.org/algebra/
Other
378 stars 69 forks source link

Remove various unnecessary modules #161

Closed erik-stripe closed 7 years ago

erik-stripe commented 7 years ago

This PR implements the proposal in #160. It changes the module structure without changing any of the core type classes (although it does remove two types used in testing: TypeTagM and Platform, which are now available to us through our catalysts test dependency).

After this change, we would have only two subprojects:

I didn't go ahead and enable MiMA on this yet because I wanted to float the idea of removing those two types that are extraneous. I can readd them if we think compatibility there is worth it. I'm also happy to verify that those two types are the only things that changed.

What do you think @johnynek? Also interested to hear from @tixxit and others on this.

johnynek commented 7 years ago

I'm +1 but let's get one more buy in.

tixxit commented 7 years ago

With the move to cats kernel, it takes the pressure off to provide such a small surface area. I'm definitely 👍 on this.