CSV Helper Converters need to use TypeConverterOptions

[Jass91]

Hi guys, I am trying to use CSV Helper to map empty cells to a DateTime? using DateTimeConverter

I realized that the converter does not use TypeConverterOptions, so, even I mapping like this:

Map(m => m.TradeTime)
    .Optional()
    .Name("TradeDate")
    .TypeConverterOption.NullValues(string.Empty)
    .TypeConverterOption.Format(dateFormats)
    .TypeConverter(new CsvHelper.TypeConversion.Test.DateTimeConverter2());

I have an error complaining about parse error, So I think DateTimeConverter should change a little as follow:

if (text == null)
{
    return base.ConvertFromString(null, row, memberMapData);
}

var assumeNull = memberMapData.TypeConverterOptions.NullValues.Contains(text);
if (assumeNull)
    return null;
// rest of the original code

Do you agree?