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.

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

Add header to generated CSV file #14

Closed
wants to merge 1 commit into from
Closed

Add header to generated CSV file #14

wants to merge 1 commit into from

Conversation

guyfawcus
Copy link
Owner

Pretty sure that this is a good idea but I'll leave this open for a while to see if anyone has any issues with it...

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 57.595% when pulling ac83049 on csv-header into 0213ed9 on master.

@guyfawcus
Copy link
Owner Author

Taken care of in ec1aeb8

@guyfawcus guyfawcus closed this Nov 3, 2017
@guyfawcus
Copy link
Owner Author

I was convinced after reading https://www.thoughtspot.com/blog/6-rules-creating-valid-csv-files (Rule 4)

@guyfawcus guyfawcus deleted the csv-header branch November 3, 2017 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants