icsv2ledger icon indicating copy to clipboard operation
icsv2ledger copied to clipboard

Documentation: --desc should be integer

Open fleimgruber opened this issue 5 years ago • 2 comments

The docs state

--desc STR            CSV column number matching description
--effective-date INT  CSV column number matching effective date

which should read

--desc INT            CSV column number matching description
--effective-date INT  CSV column number matching effective date

Or even better, specify CSV column by name since ordering of columns might be arbitrary depending on tools prior in the CSV transformation chain.

fleimgruber avatar Feb 21 '20 13:02 fleimgruber

This is STR because one can specify a comma-separated list of column indices.

areynoua avatar Jul 12 '22 17:07 areynoua

Thanks for checking back, has been a while.

This is STR because one can specify a comma-separated list of column indices.

This should be mentioned in the docs then? E.g.

--desc STR            CSV column number or comma-separated list of column indices matching description

and I am wondering how a description can be taken from multiple columns? Does it just concatenate all the column values then?

fleimgruber avatar Jul 15 '22 08:07 fleimgruber