CsvReader.GetRecords with ShouldSkipRecord may skip the header row

[ryst]

Describe the bug
When using CsvConfiguration.ShouldSkipRecord, CsvReader.GetRecords may skip the header row.

To Reproduce

public class CsvData
{
    public string Name { get; set; }
}

var csvContent = "Name\nArnold\nAndrew\nBetty\nFrank";

using var stream = new MemoryStream(Encoding.UTF8.GetBytes(csvContent));
using var reader = new StreamReader(stream);

var configuration = new CsvConfiguration(CultureInfo.InvariantCulture)
{
    ShouldSkipRecord = shouldSkipRecordArgs => !shouldSkipRecordArgs.Row[0].StartsWith("A")
};

var csvReader = new CsvReader(reader, configuration);
var records = csvReader.GetRecords<CsvData>().ToList();

Expected behavior
GetRecords should return two records, Arnold and Andrew. However, the header row is skipped and "Arnold" becomes the header. Then a HeaderValidationException is thrown for missing header with name "Name". ShouldSkipRecord should not apply to the header row.

Additional context
If I add a ClassMap to map index 0 to the Name property, I won't get the HeaderValidationException, but the GetRecords will only return one record, Andrew.

class CsvDataMap : ClassMap<CsvData>
{
    public CsvDataMap()
    {
        Map(m => m.Name).Index(0);
    }
}

csvReader.Context.RegisterClassMap<CsvDataMap>();

[omerbenda]

Hi! #2372 Should fix this issue.
I do want to emphasize that this behavior might be intended, perhaps to skip over starting lines before the actual data, as the first test in the ShouldSkipRecordTests class utilizes this to skip over a blank line.