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

documentation improvements on getline_hr_all () #23

Merged
merged 1 commit into from Aug 27, 2016

Conversation

Projects
None yet
2 participants
@garu
Contributor

garu commented May 30, 2015

Hi again!

This PR is based on the diff provided in this RT ticket, and reminds the user that getline_hr_all () needs column_names () just like getline_hr () does.

Hope it helps! Thanks!

@charsbar

This comment has been minimized.

Show comment
Hide comment
@charsbar

charsbar Apr 21, 2016

Collaborator

@makamaka, please merge this. This is a doc patch.

Collaborator

charsbar commented Apr 21, 2016

@makamaka, please merge this. This is a doc patch.

@charsbar charsbar merged commit cb32605 into makamaka:master Aug 27, 2016

@charsbar

This comment has been minimized.

Show comment
Hide comment
@charsbar

charsbar Aug 27, 2016

Collaborator

Thanks!!!! (it's getting harder and harder to keep track of how many !s I add to the previous ticket...)

Collaborator

charsbar commented Aug 27, 2016

Thanks!!!! (it's getting harder and harder to keep track of how many !s I add to the previous ticket...)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment