-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Add Intro to OpenTelemetry Java page #5471
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Click the link in the "Managed Dependencies" column to see a list of the | ||
artifacts managed by the BOM. | ||
|
||
| Description | Repository | Group ID | Artifact ID | Current Version | Managed Dependencies | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The "Managed Dependencies" column of this table is awesome. It links to the current version of each BOM's pom.xml
, allowing the user to see the list of managed dependencies at a glance, guaranteed to be up to date.
different use cases, listed below in order of increasing scope. We highly | ||
recommend using a BOM. Because the BOMs are hierarchical, adding dependencies on | ||
multiple BOMs is not recommended, as it is redundant and can lead unintuitive | ||
dependency version resolution. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding dependencies on multiple BOMs is not recommended
This is a new an important piece of advice. Choose your BOM wisely!
@@ -14,74 +14,3 @@ weight: 18 | |||
--- | |||
|
|||
{{% docs/languages/index-intro java /%}} | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All this content moves to intro.md
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me overall! thanks once again
Ping @open-telemetry/docs-maintainers. I think this is good to merge. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for all the work here.
Part of #5211.
Add new "Intro to OpenTelemetry Java" page, introducing key aspects of the OpenTelemetry Java ecosystem and establishing basic vocabulary to help route users to the right pages for additional details.
Preview available here: https://deploy-preview-5471--opentelemetry.netlify.app/docs/languages/java/intro/