I think this is hard to read because it's parsing certain config with conventions that aren't clear from the names. I'm confused by file_to_read being an int or a list, I'd expect a filename.
Probably a dataclass or a pydantic class (if we're willing to add the dependency) would make this more readable. But that may be out of scope for now.
Originally posted by @MishaVeldhoen in #31 (comment)
I think this is hard to read because it's parsing certain config with conventions that aren't clear from the names. I'm confused by file_to_read being an int or a list, I'd expect a filename.
Probably a dataclass or a pydantic class (if we're willing to add the dependency) would make this more readable. But that may be out of scope for now.
Originally posted by @MishaVeldhoen in #31 (comment)