dilshod / xlsx2csv

Convert xslx to csv, it is fast, and works for huge xlsx files
MIT License
1.68k stars 303 forks source link

Request re-introduction of multi-character sheet delimiters #284

Open realhalo opened 3 months ago

realhalo commented 3 months ago

https://github.com/dilshod/xlsx2csv/blob/bed9ef1812f426be5cf6d696af0459175eee2c0a/xlsx2csv.py#L1196

In previous versions it was allowed to have "-s multi-character-string" as a sheet delimiter but has since rejected as invalid in more recent versions, any reason to not allow it like was previously supported?

Very useful script and it has made my life easier for some work tasks, keep on keepin' on either way.

dilshod commented 3 months ago

I don't see any reason why this restriction is needed, it was added 9 years ago

realhalo commented 3 months ago

The version checked into our repo was from 2011 heh, we didn't have an issue with it until now and whatever you did between then and now fixed it so... keep up the good work.