Skip to content
This repository has been archived by the owner on May 24, 2023. It is now read-only.

Typo correction for #62 #61

Closed
wants to merge 1 commit into from
Closed

Typo correction for #62 #61

wants to merge 1 commit into from

Conversation

joshuaprunier
Copy link

No description provided.

@xaprb
Copy link
Contributor

xaprb commented Dec 13, 2014

It's not a typo. http://golang.org/pkg/database/sql/#Rows.Err

@xaprb xaprb closed this Dec 13, 2014
@arnehormann
Copy link
Contributor

@xaprb it's not a typo, but it can be misunderstood as one. The text directly below talks about rows.Close() and it says "Consider the following code" above - which doesn't contain rows.Close().

@xaprb xaprb reopened this Dec 14, 2014
@xaprb
Copy link
Contributor

xaprb commented Dec 14, 2014

OK. Probably got all mixed up while I was rewriting a bunch of this. Will take a look later.

@joshuaprunier
Copy link
Author

Ah, I think I see now. It looks like it may become 2 sections; Errors From Iterating Resultsets and Errors From Close. Bad pull request. My apologies.

@xaprb xaprb mentioned this pull request Dec 16, 2014
@xaprb xaprb changed the title Typo correction Typo correction for #62 Dec 16, 2014
@xaprb xaprb self-assigned this Dec 16, 2014
@xaprb
Copy link
Contributor

xaprb commented Dec 16, 2014

I pushed a branch with my proposed changes in #63. Review

@joshuaprunier
Copy link
Author

Looks good to me. I was previously unaware of rows.Err() so I need to go back and modify some code. Thanks again for a great tutorial!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants