search

ZJONSSON / node-etl

npm install etl
233 stars 49 forks source link

__line #118

Closed bernardoadc closed 3 years ago

bernardoadc commented 3 years ago

Expected Behavior / Situation

.pipe(csv({ originalLine: false }))
/*
a;b;c            [
1;2;3   --->     {a:1, b:2, c:3},
4;5;6               {a:4, b:5, c:6},
                   ]
*/

Actual Behavior / Situation

.pipe(csv())
/*
a;b;c            [
1;2;3   --->     {a:1, b:2, c:3, __line: 2},
4;5;6               {a:4, b:5, c:6, __line: 3},
                   ]
*/

Modification Proposal

To have an option ("originalLine" in example) to inhibit the addition of "__line" property to the resulting objects. It's bad for SQL insert, for example.

bernardoadc commented 3 years ago

Actually, it's covered in #109