Skip to content
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

Updated OpenCensus and Stackdriver Monitoring dependencies. #203

Merged
merged 8 commits into from
Nov 28, 2019

Conversation

jkohen
Copy link
Contributor

@jkohen jkohen commented Nov 27, 2019

This pulled in a bunch of indirect dependencies.

@jkohen jkohen self-assigned this Nov 27, 2019
@jkohen jkohen requested a review from a team November 27, 2019 21:04
@ghost ghost requested review from igorpeshansky and removed request for a team November 27, 2019 21:04
@@ -104,7 +104,7 @@ func init() {
panic(err)
}
if err := view.Register(
ocgrpc.DefaultClientViews...
ocgrpc.DefaultClientViews...,
Copy link
Member

Choose a reason for hiding this comment

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

Just curious, how is this change related to the rest of this PR?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The PR that introduced that line didn't format the code. My bad!

@jkohen jkohen requested a review from igorpeshansky November 27, 2019 23:04
@jkohen jkohen changed the title Updated OpenCensus dependency. Updated OpenCensus and Stackdriver Monitoring dependencies. Nov 27, 2019
This reverts commit fdb1716.
@jkohen
Copy link
Contributor Author

jkohen commented Nov 28, 2019

@igorpeshansky ready for review. I didn't get go mod tidy to pass without more effort. Next time.

Copy link
Member

@igorpeshansky igorpeshansky left a comment

Choose a reason for hiding this comment

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

LGTM :shipit:

@jkohen jkohen merged commit fc16d1f into master Nov 28, 2019
@jkohen jkohen deleted the jkohen-oc_vendor branch November 28, 2019 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants