ImperialCollegeLondon / pycsvy

Python reader/writer for CSV files with YAML header information
BSD 3-Clause "New" or "Revised" License
7 stars 8 forks source link

WIP: Class version #126

Open dalonsoa opened 4 weeks ago

dalonsoa commented 4 weeks ago

This PR suggest some exploratory work implementing the same functionality (the readers in this case) using classes and subclasses.

The reason for this change is:

No need to do a thorough review, just gathering some feedback on the approach, pros, cons and things to consider.

AdrianDAlessandro commented 3 weeks ago

I think this approach helps with keeping things consistent and not repeating code. There could be a more more functional way to do this, but I think this is maybe clearer than that would be.