stephan-tolksdorf / fparsec

A parser combinator library for F#
523 stars 45 forks source link

Naming `ParseResult` fields. #105

Open smoothdeveloper opened 10 months ago

smoothdeveloper commented 10 months ago

In using the library, I found that I want to deconstruct ParseResult using named fields rather than positional; also, it is not considered a good practice to leave DU fields unnamed.

So I'm suggesting we look at those changes, and consider if they are aligned with the style of the library.

If so, I can spend some time in reviewing the documentation and doing similar stylistic adjustment in some of the code examples, if desired.