well-typed / optics

Optics as an abstract interface
375 stars 24 forks source link

Clarify makeFieldLabels prefix requirements #472

Open adamgundry opened 2 years ago

adamgundry commented 2 years ago

Talking to @edsko I think the current distinction between makeFieldLabels and makeFieldLabelsNoPrefix is a bit non-obvious. In particular, if a user uses makeFieldLabels on a datatype that has no prefixes, it will silently succeed but generate no LabelOptic instances.

I think we should: