search

CyrusOfEden / CSV.js

A simple, blazing-fast CSV parser and encoder. Full RFC 4180 compliance.
MIT License
1.54k stars 95 forks source link

minor doc fix #23

Closed alexmasselot closed 10 years ago

alexmasselot commented 10 years ago

it looks to me that the is not much cellDelimiter in the js lib, but 'delimiter' is more (thanks a lot for the package. it's cool and fast)

CyrusOfEden commented 10 years ago

Actually, for v3.5 and up the preferred option key is cellDelimiter and lineDelimiter instead of delimiter and line respectively so as to be more explicit. However, you may still use the older methods, but those will probably be phased out come v4.

alexmasselot commented 10 years ago

thanks. And that makes sense. my bower.json was pointing to ~3.3.0 and that was the problem ;) Alex

On Thu, Sep 18, 2014 at 2:39 PM, Kash Nouroozi notifications@github.com wrote:

Closed #23 https://github.com/knrz/CSV.js/pull/23.

— Reply to this email directly or view it on GitHub https://github.com/knrz/CSV.js/pull/23#event-167437692.

Alexandre Masselot

http://alexandre-masselot.blogspot.com/ http://mtroopgoeswest.blogspot.com/

CyrusOfEden commented 10 years ago

I'm following semver so you should be safe specifying ~3.3 if you really want.

alexmasselot commented 10 years ago

thanks

On Thu, Sep 18, 2014 at 3:31 PM, Kash Nouroozi notifications@github.com wrote:

I'm following semver so you should be safe specifying ~3.3.

— Reply to this email directly or view it on GitHub https://github.com/knrz/CSV.js/pull/23#issuecomment-56112820.

Alexandre Masselot

http://alexandre-masselot.blogspot.com/ http://mtroopgoeswest.blogspot.com/