Skip to content
This repository was archived by the owner on Dec 23, 2023. It is now read-only.

add spring starter package for opencensus spring integration.#1934

Merged
rghetia merged 3 commits intocensus-instrumentation:masterfrom
rghetia:spring_starter
Jun 12, 2019
Merged

add spring starter package for opencensus spring integration.#1934
rghetia merged 3 commits intocensus-instrumentation:masterfrom
rghetia:spring_starter

Conversation

@rghetia
Copy link
Copy Markdown
Contributor

@rghetia rghetia commented Jun 11, 2019

No description provided.

Copy link
Copy Markdown
Contributor

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

LGTM, assuming there will be other PRs adding the actual implementation.

Comment thread contrib/spring_starter/README.md Outdated
opencensus.spring.trace.propagation = TRACE_PROPAGATION_B3
```

[travis-image]: https://travis-ci.org/census-instrumentation/opencensus-java.svg?branch=master
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These links seem to be duplicated with the ones at the bottom.

Copy link
Copy Markdown
Contributor

@songy23 songy23 left a comment

Choose a reason for hiding this comment

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

Consider adding package-info.java or an empty class, otherwise build will fail.

@rghetia rghetia merged commit f0c0ca2 into census-instrumentation:master Jun 12, 2019
@rghetia rghetia deleted the spring_starter branch June 12, 2019 00:26
@rghetia
Copy link
Copy Markdown
Contributor Author

rghetia commented Jun 12, 2019

@songy23 you can release it now.

@songy23
Copy link
Copy Markdown
Contributor

songy23 commented Jun 12, 2019

Got it, let's do it tomorrow morning :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants