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

Ability to read line by line file with header #40

Closed
Dogzik opened this issue Apr 20, 2020 · 3 comments
Closed

Ability to read line by line file with header #40

Dogzik opened this issue Apr 20, 2020 · 3 comments
Labels
enhancement New feature help wanted Extra attention is needed

Comments

@Dogzik
Copy link

Dogzik commented Apr 20, 2020 •
edited
Loading

It is very common to have csv headers in csv files. However not always I can or need to store the whole file in memory. Right now there is possibility either to read file line by line or read all data with header. It would be really useful to be able to do both at the same time
@doyaaaaaken doyaaaaaken added the enhancement New feature label Apr 22, 2020
@doyaaaaaken
Copy link
Owner

Thank you so much for your valuable advice!

I agree your opinition, and I think we need below method.
fun readWithHeaderAsSequence(): Sequence<Map<String, String>>

@doyaaaaaken doyaaaaaken added the help wanted Extra attention is needed label Apr 22, 2020
@Dogzik
Copy link
Author

Dogzik commented Apr 22, 2020

Yeah, sounds just as I imagined it

@blackmo18 blackmo18 mentioned this issue May 20, 2020
Closed
@doyaaaaaken doyaaaaaken mentioned this issue May 23, 2020
Merged
@doyaaaaaken
Copy link
Owner

Great job @blackmo18 ! Changes are included in version 0.8.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@doyaaaaaken @Dogzik