hvalev/py-markdown-table

Tiny python library with zero dependencies which allows you to generate formatted markdown tables

py-markdown-table's Issues

Padding around column seperator

Currently there is no whitespace around the column separator which is as far as I know totally conforming to the standard but (in my personal opinion) a little bit hard to read in the raw view (and I have never seen it that way).

e.g.

test = [{"A": "row1_A", "B": "row1_B", "C": "row1_C"},
        {"A": "row2_A", "B": "row2_B", "C": "row2_C"},
        {"A": "row3_A", "B": "row3_B", "C": "row3_C"}]

tabletest = mdT.markdownTable(test).setParams(row_sep="markdown", 
                                              padding_weight="right", 
                                              quote=False).getMarkdown()
print(tabletest)

Output:

|A     |B     |C     |
|------|------|------|
|row1_A|row1_B|row1_C|
|row2_A|row2_B|row2_C|
|row3_A|row3_B|row3_C|

I would prefer:

| A      | B      | C      |
| ------ | ------ | ------ |
| row1_A | row1_B | row1_C |
| row2_A | row2_B | row2_C |
| row3_A | row3_B | row3_C |

Do you accept a PR for this (maybe also as an optional setting) or do you want it to keep it the way it is?

Please add a way to remove padding entirely.

Option for title case headers or custom headers

It would be nice have the table headers be capitalized via an option without needing to change all the dict keys to capitals. Additionally, it might be nice to allow custom headers different than the dict keys. One use case for this would be a boolean column where you may want to trail the column name with a question mark, for example. Another use case would be if you want spaces in the header.

Value Error for when adding links in the table.

If we create a link in a table [link] (url), as URLs are long & single string, Its pretty much always throws Value Error when clearly there is no need.
https://github.com/hvalev/py-markdown-table/blob/main/py_markdown_table/markdown_table.py#L276

Posible solution could be to detect url & ignore them, but i would say completely remove the Value error check also will also fine