Skip to content

Conversation

ericharmeling
Copy link
Contributor

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@cockroach-teamcity
Copy link
Member

Copy link
Contributor

@ianjevans ianjevans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Everything works, I'd just eliminate the first 2 step sections.

<section class="filter-content" markdown="1" data-scope="secure">

## Step 3. Create the `maxroach` user and `bank` database
## Step 3. Start CockroachDB
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe running go mod init below eliminates the need for Steps 1 & 2. All the library deps are resolved at that point.

Copy link
Contributor Author

@ericharmeling ericharmeling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ianjevans TFTR!

Reviewable status: :shipit: complete! 0 of 0 LGTMs obtained (waiting on @ianjevans)


v20.2/build-a-go-app-with-cockroachdb.md, line 35 at r1 (raw file):

Previously, ianjevans (Ian Evans) wrote…

I believe running go mod init below eliminates the need for Steps 1 & 2. All the library deps are resolved at that point.

Done.

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.

3 participants