csv icon indicating copy to clipboard operation
csv copied to clipboard

Only remove the unescape prefix character when parsing

Open jgmchan opened this issue 1 year ago • 1 comments

This PR addresses

  • Currently the unescape_formulas option when parsing will remove any first character of a cell as long as the second character is part of the @escape_formula_start. This causes issues when the data is something like this:

    "A-1","B+2","C=3","'@D"

    which should parse to this:

    ["A-1", "B+2", "C=3", "@D"]

    but parses to this instead:

    ["-1", "+2", "=3", "@D"]

    This change will check that the first character is actually the escape character we want before removing it.

  • Slight fix up of the documentation

jgmchan avatar Sep 18 '24 04:09 jgmchan

The failing tests doesn't seem to be related to my change and I can't figure out how to kick it off again. Any ideas @beatrichartz ?

jgmchan avatar Sep 24 '24 05:09 jgmchan

Looking good, merged. Thanks!

beatrichartz avatar Jan 02 '25 09:01 beatrichartz

Published in 3.2.2

beatrichartz avatar Jan 02 '25 10:01 beatrichartz