Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
Add | (pipe) as a potential CSV cell separator. #1348
Conversation
A number of financial file format standards make use of the pipe as a CSV cell separator
|
Hi guys |
|
Even though Excel doesn't normally handle pipe separators, this PR is mostly ok. It's better to set the weights to 4/3/2/1. Related questions:
|
* upstream/master: TSV Files can start with tab characters bug: Remove white spaces due to html tags (#1622) fixing some typos in the documentation initial release of S [ci skip] Fix #1244 Fix issue #1536 version bump 0.15.1 version bump 0.15.0: mini build version bump 0.14.5: XLS grind README Fix (fixes #1546) version bump 0.14.4 travis config to_csv skipHidden corner case (fixes #1508) version bump 0.14.3: formula niggle (closes #1388) version bump 0.14.2: comment xml (fixes #1468) README use typed array (fixes #1362) version bump 0.14.1: AutoFilter issues
|
Thanks for your reply
|
|
Hello again when parsing (valid) csv files with pipe as separator, sometimes XSLX does not detect the pipe as the separator even though it is by far the most frequent of the separators
will fail to detect the pipe whereas
would! i tried to track the issue in the code but lost myself :( |

Formed in 2009, the Archive Team (not to be confused with the archive.org Archive-It Team) is a rogue archivist collective dedicated to saving copies of rapidly dying or deleted websites for the sake of history and digital heritage. The group is 100% composed of volunteers and interested parties, and has expanded into a large amount of related projects for saving online and digital history.

Hi
A number of financial file format standards make use of the pipe as a CSV cell separator
it would be nice if you could accept this feature
I used 0 as weight thinking it was the lowest priority (i hope i didn't misread the code)
let me known if you need more info/some changes to accept the pull request
Thanks
David