quentinsf / icsv2ledger

Interactive importing of CSV files to Ledger
196 stars 70 forks source link

Convert the decimal separator of numbers #61

Closed vanicat closed 10 years ago

vanicat commented 10 years ago

It use two option, one declaring the what separator is used in csv, the other in ledger. If both or none are set, no conversion is needed. Otherwise a conversion is needed.

I wonder if this could be made cleared by defining a --convert-decimal-comma-to-decimal-dot and a --convert-decimal-dot-to-decimal-comma option. It seem more natural in icsv2ledger to use declarative option as I did, but I'm not completely sure of myself.

petdr commented 10 years ago

Looks fine, thanks.