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

Update README to document jaeger-thrift exporter support. #1882

Merged

Conversation

breedx-splk
Copy link
Contributor

This is a follow-up to #1868. It adds configuration information for jaeger thrift to the README.

@anuraaga anuraaga merged commit 4af2655 into open-telemetry:master Dec 15, 2020
@trask
Copy link
Member

trask commented Dec 15, 2020

any thoughts how mark README PRs for new features not for merging until release? (not a big deal, best effort 😄)

@iNikem
Copy link
Contributor

iNikem commented Dec 15, 2020

Should we switch branching model to "branch per future minor release"?

@anuraaga
Copy link
Contributor

Ah shoot - sorry about that :( The suspiciously large number of green checks should have warned me :P

I think a branch that only has doc updates could make sense since it should be trivial to keep up to date with mainline by just merging it in as needed.

We could also just have a convention of doc PRs being kept in draft mode

anuraaga added a commit that referenced this pull request Dec 15, 2020
anuraaga pushed a commit that referenced this pull request Dec 15, 2020
anuraaga added a commit that referenced this pull request Dec 15, 2020
bhautikpip pushed a commit to bhautikpip/opentelemetry-java-instrumentation that referenced this pull request Dec 18, 2020
bhautikpip pushed a commit to bhautikpip/opentelemetry-java-instrumentation that referenced this pull request Dec 18, 2020
anuraaga pushed a commit that referenced this pull request Dec 18, 2020
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.

None yet

6 participants