Hey @idg10 , we're using this library (at @Dualog) and wanted to contribute back.

Could you clarify if you would like to have all of the ArgumentException s to reference the line input parameter? It's the only argument for the method but some of the argumentExceptions are being thrown on a subset of that line

See fredeil@2655552

Yes, I just meant that the relevant throws should use nameof(line) as the paramName argument when constructing the ArgumentException. (So exactly what you've done in that commit.)

Thanks for offering to do this. While you're in there would you mind adding an <exception> tag to the relevant XML doc comments, so it's clear to anyone using this library that this is how we report unparseable data?

Sure, that was my next question.

No more documentation needed outside of that? @idg10

Nothing I'm aware of, but if you feel it's missing something feel free either to add more, or to open another issue describing what's missing.