commercialhaskell / rio

A standard library for Haskell
Other
838 stars 54 forks source link

cartesianProduct is missing in RIO.Set #177

Open hanjoosten opened 5 years ago

hanjoosten commented 5 years ago

Is there any reason why cartesianProduct has not been implemented in RIO.Set?

snoyberg commented 5 years ago

Looks like it's just a relatively recent addition to the API. There may be a problem with including it, since we'd have to avoid the export for older versions of containers. But it looks like we already do plenty of conditional reexports for containers. Want to send a PR to add the reexport?