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

Fix reference to 'db' var #103

Merged
merged 1 commit into from
Mar 7, 2019
Merged

Fix reference to 'db' var #103

merged 1 commit into from
Mar 7, 2019

Conversation

eliben
Copy link
Contributor

@eliben eliben commented Mar 7, 2019

In the rest of the example db is used, not Db. Moreover, I think that a simple example here could help clarify the terminology, because it gets a bit murky between db (object of type DB) and tx (object of type Tx). Something like:

// Create an object of type Tx
tx, err := db.Begin()
...

In the rest of the example `db` is used, not `Db`. Moreover, I think that a simple example here could help clarify the terminology, because it gets a bit murky between `db` (object of type `DB`) and `tx` (object of type `Tx`). Something like:

```
// Create an object of type Tx
tx, err := db.Begin()
...
```
@xaprb xaprb merged commit ccce84f into VividCortex:gh-pages Mar 7, 2019
@xaprb
Copy link
Contributor

xaprb commented Mar 7, 2019

@eliben I'd like to add you as a collaborator, is that okay? You have good judgment and you should just make the changes you see fit. Do you have two-factor auth enabled on GitHub? it's part of our corporate security policy.

@eliben
Copy link
Contributor Author

eliben commented Mar 7, 2019

@eliben I'd like to add you as a collaborator, is that okay? You have good judgment and you should just make the changes you see fit. Do you have two-factor auth enabled on GitHub? it's part of our corporate security policy.

I don't think that will be necessary. I was just reading through the tutorial and I'm done now :-)

Thanks for publishing it

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.

2 participants