Use constructor property promotion

This looks like a good change to me. I am also curious if we could take this opportunity to add explicit argument types to the constructor arguments. I will make that change and push it up.

The PHPStan failure is related to an upstream deprecation. I have created 📌 Replace deprecated usage of League\Csv\ByteSequence::BOF_UTF8 Needs review to address that.

Merged in the change from #3455761 (#4) and the pipeline passes now without failure. Back to you now.

I didn't add the argument types out of an abundance of caution, but you're much more familiar with the module that I am.

I didn't add the argument types out of an abundance of caution, but you're much more familiar with the module that I am.

Should be safe to add at PHP 7.4 and above! We should be good to add types throughout the module as we improve the code.

Automatically closed - issue fixed for 2 weeks with no activity.