Clever / csvlint

library and command line tool that validates a CSV file
Apache License 2.0
189 stars 20 forks source link

Please add support for delimiter "pipe" which looks like "|" Also: colon and semicolon. #2

Closed rynes closed 9 years ago

rynes commented 9 years ago

The pipe delimiter "|" is very popular and is the default for applications like sqlite which I use. Please add the pipe to csvlint as a delimiter. Also, next in line are probably colon (":") and then semicolon (";"). Adding them when you are in there might save time in the future. Since I don't have a go compiler, please update the windows executable "csvlint.exe".

azylman commented 9 years ago

I'm closing this in favor of #4, let me know what you think. I'm not sure when I'll get a chance to do it - feel free to send a PR :)

rynes commented 9 years ago

Yes that’s fine.

From: Alex Zylman [mailto:notifications@github.com] Sent: Monday, December 29, 2014 1:27 PM To: Clever/csvlint Cc: Philip Rynes Subject: Re: [csvlint] Please add support for delimiter "pipe" which looks like "|" Also: colon and semicolon. (#2)

I'm closing this in favor of #4 https://github.com/Clever/csvlint/issues/4 , let me know what you think. I'm not sure when I'll get a chance to do it - feel free to send a PR :)

— Reply to this email directly or view it on GitHub https://github.com/Clever/csvlint/issues/2#issuecomment-68293232 . https://github.com/notifications/beacon/AIRXVr2f7X_zEifyjOJZi6jreIwKH45cks5ncaKSgaJpZM4DLLuB.gif

schimmy commented 9 years ago

Hey @rynes, we've added support with https://github.com/Clever/csvlint/pull/5, released as https://github.com/Clever/csvlint/releases/tag/v0.2.0. Please let me know how the change is working for you, and thanks for using csvlint!