purescript-deprecated / purescript-lens

PureScript implementation of lens
78 stars 15 forks source link

add Iso type and constructor #22

Closed julien-truffaut closed 8 years ago

julien-truffaut commented 9 years ago

first purescript commit ^^

paf31 commented 9 years ago

Have you seen https://github.com/joneshf/purescript-optic/blob/master/src/Optic/Iso.purs?

julien-truffaut commented 9 years ago

no, I assumed all optics where defined in this package. We can drop this PR then, by curiosity why Lens and Prism are in one package and Iso and Traversal in another?

garyb commented 8 years ago

They used to be one large package, but I think Hardy split them up to stop lens becoming a monolithic thing that pulls in tons of dependencies like lens does in Haskell perhaps.

There's also https://github.com/purescript-contrib/purescript-profunctor-lenses now, which does have all of the stuff together again, but is based on a slightly different lens formulation.