search

DavidArno / SuccincT

Discriminated unions, pattern matching and partial applications for C#
MIT License
266 stars 15 forks source link

Fix need to expose UnionCreator<..> classes as public by using interfaces #24

Closed DavidArno closed 7 years ago

DavidArno commented 7 years ago

The following types:

UnionCreator<T1, T2>
UnionCreator<T1, T2, T3,>
UnionCreator<T1, T2, T3, T4>

All contain the weasel words:

Factory class created by Union{...}.Creator(). Whilst this is a public class (as the user needs access to Create()), it has an internal constructor as it's intended for union creation internal usage only.

In all three cases, these should be able to be made internal, with a public facing interface, which is the type returned by Union<...>.Creator().

DavidArno commented 7 years ago

Included in the v3.0.0 release.