Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CSV shouldn't call compact #568

Closed
ghost opened this issue Sep 15, 2017 · 0 comments · Fixed by #574
Closed

CSV shouldn't call compact #568

ghost opened this issue Sep 15, 2017 · 0 comments · Fixed by #574
Labels
bug
Milestone
v4.0.2

Comments

@ghost
Copy link

ghost commented Sep 15, 2017

When CSV calls compact it removes nils from the array. However if you leave a blank entry this removes the blank thus effectively shifts the remaining columns left. The user can call compact as necessary but CSV must return the data as is.
@ghost ghost added the bug label Sep 15, 2017
@ghost ghost self-assigned this Sep 15, 2017
@ghost ghost mentioned this issue Sep 26, 2017
Merged
@ghost ghost closed this as completed in #574 Sep 26, 2017
@ghost ghost added this to the v4.0.2 milestone Sep 29, 2017
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
v4.0.2
Development

Successfully merging a pull request may close this issue.

Don't compact csv lines BallAerospace/COSMOS
0 participants