b-souty / html2dict Goto Github PK

I will add docstrings to the current code in order to improve maintainability. Going forward, any new function should have a docstring to start with.

edit: Can someone please review my PR #3

Currently the returned table are stored in a dictionary where each key is "table_n" where n is the number of the table on the page (i.e. {'table_0': [], 'table_1': [], ...})

If the caption tag is present in the table, the caption text should be used instead of "table_n".

While writing the docstrings, I realised it was a nightmare to describe the structure of a dictionary representing a table.

It might be easier to create a 'Table' class with an attribute for each type of table (raw html table, basic text table, rich formatted table, ...) . Or maybe even subclasses for each type. I'm not sure at this point.

I am open to suggestion.