Skip to content

vendor: bump cockroach-go#109

Merged
irfansharif merged 1 commit intomasterfrom
200710.bump-cockroach-go
Jul 15, 2020
Merged

vendor: bump cockroach-go#109
irfansharif merged 1 commit intomasterfrom
200710.bump-cockroach-go

Conversation

@irfansharif
Copy link
Copy Markdown
Contributor

@irfansharif irfansharif requested a review from rafiss July 10, 2020 23:19
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@irfansharif irfansharif force-pushed the 200710.bump-cockroach-go branch from cfe3434 to c33ac73 Compare July 10, 2020 23:56
Copy link
Copy Markdown
Contributor

@rafiss rafiss left a comment

Choose a reason for hiding this comment

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

thanks for the update! small request

Reviewable status: 0 of 16 files reviewed, 1 unresolved discussion (waiting on @irfansharif and @rafiss)


Gopkg.toml, line 79 at r1 (raw file):

[[constraint]]
  name = "github.com/cockroachdb/cockroach-go"
  revision = "b7c8285b7ed3b7e117851f11bdb56ad33cc16320"

would you mind specifying it using version = v2.0.3 instead? (i just pushed a new tag to cockroach-go)

@irfansharif irfansharif force-pushed the 200710.bump-cockroach-go branch from c33ac73 to 8dd4206 Compare July 14, 2020 23:15
Picks up cockroachdb/cockroach-go#81. Cleans up
test code along the way to work with latest cockroach-go
(testserver.NewTestServer already starts the server it seems, there's no
need to start it again).
@irfansharif irfansharif force-pushed the 200710.bump-cockroach-go branch from 8dd4206 to f9b1147 Compare July 14, 2020 23:16
Copy link
Copy Markdown
Contributor Author

@irfansharif irfansharif left a comment

Choose a reason for hiding this comment

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

Reviewable status: 0 of 16 files reviewed, 1 unresolved discussion (waiting on @rafiss)


Gopkg.toml, line 79 at r1 (raw file):

Previously, rafiss (Rafi Shamim) wrote…

would you mind specifying it using version = v2.0.3 instead? (i just pushed a new tag to cockroach-go)

Done.

Copy link
Copy Markdown
Contributor

@rafiss rafiss left a comment

Choose a reason for hiding this comment

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

lgtm!

Reviewed 14 of 16 files at r1, 2 of 2 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @rafiss)

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