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

Add Jaeger support to README and fix links #383

Merged
merged 1 commit into from
Jan 30, 2018

Conversation

rakyll
Copy link
Contributor

@rakyll rakyll commented Jan 30, 2018

No description provided.

Copy link
Member

@odeke-em odeke-em left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @rakyll, I've just added a small question.

@@ -255,7 +256,7 @@ An example logger exporter is below:

type exporter struct{}

func (e *exporter) Export(vd *stats.ViewData) {
func (e *exporter) ExportView(vd *stats.ViewData) {
Copy link
Member

Choose a reason for hiding this comment

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

Did we forget to perhaps do this in the internal/readme/source.md file too? I don't know how the embedmd stuff works :)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I assumed someone changed README rather than internal/readme/source.md.

@rakyll rakyll merged commit 7e6c5d7 into census-instrumentation:master Jan 30, 2018
@rakyll rakyll deleted the readme branch January 30, 2018 21:55
bogdandrutu pushed a commit to bogdandrutu/opencensus-go that referenced this pull request Feb 15, 2019
Adding a timeout to tests to help cache any test that maybe deadlocking with certain frequency.
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

2 participants