Update to allow csv parsing

Whatapalaver / jsonl-parsing-cli

0 stars 3 forks source link

Update to allow csv parsing #5

Open Whatapalaver opened 4 years ago

Whatapalaver commented 4 years ago

This branch is set upto to parse newline delimited JSON, can we add the option to process for a range of file types or have branches for each?

smowafy commented 4 years ago

What's the best way to go about it?

For example:

A command line argument as the explicit way
Try to deduce from the file extension
Last fallback: Try parsing the supported formats till one succeeds

Or did you have another plan in mind already?

Whatapalaver commented 4 years ago

I think it would be nice to deduce from the file extension. Although I am not opposed to a command line flag which I think is preferable to the final option you proposed.

smowafy commented 4 years ago

Then the last fallback will be throwing an error stating that the format is not supported then for example?

Also whenever the plan is agreed upon I can take over this issue if possible

Whatapalaver commented 4 years ago

That’s fine by me - by all means take on the issue.