Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature request: CSV (or DSV) rendering #79

Open
e13mort opened this issue Feb 19, 2024 · 1 comment
Open

Feature request: CSV (or DSV) rendering #79

e13mort opened this issue Feb 19, 2024 · 1 comment

Comments

@e13mort
Copy link

e13mort commented Feb 19, 2024

Thank's for the great library. I use it in some of my projects and very happy with results (especially combined with Mosaic).

In one of my projects I've faced with requirement to output data as CSV along with human-readable tables. I've came up with my
own solution (not a rocket science) and got thinking that there could be some CSV related API in Picnic.

  • CSV (or more general DSV) format has no different in meaning from ASCII tables
  • Such new rendering API could benefit from existing Mosaic API (rows, columns, cell, columnSpan)
  • On the other side new API could contradict the single responsibility principle of the library (only ASCII tables)

So my questions are:

  • what do you think about supporting CSV?
  • if it works will you add that API in some future?
  • may I help with this?

Thank you

@JakeWharton
Copy link
Owner

I don't think it makes sense to include directly, but rather as a companion library. I personally don't use CSV or DSV and don't feel qualified to maintain such a thing. I made the mistake in my last table library of adding support for ResultSet conversion for databases, and regretted it more than once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants