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.

Sign up for GitHub

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

Refactor java SDK and configuration #4966

Merged
merged 22 commits into from
Sep 17, 2024
Merged

Refactor java SDK and configuration #4966

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
7c9b308
wip
jack-berg Aug 2, 2024
8e66d72
Rework sdk and config organization
jack-berg Aug 5, 2024
35cf9e4
Delete exporter, sampling, resources pages
jack-berg Aug 5, 2024
c37779e
Formatting
jack-berg Aug 5, 2024
22fbd57
revert unrelated
jack-berg Aug 5, 2024
bbc633c
PR feedback
jack-berg Aug 6, 2024
2b4ed4c
Remove usages of that
jack-berg Aug 7, 2024
bb99974
PR feedback
jack-berg Aug 7, 2024
a53743d
Rebase, upgrades Docsy
chalin Aug 9, 2024
b3ce1f3
Use section-relative paths
chalin Aug 9, 2024
e9e96ac
Fix htmltest regex
chalin Aug 9, 2024
0093e32
Fix warnings: use path for OTel spec reference not URL
chalin Aug 9, 2024
e8099d3
Fix spelling
chalin Aug 9, 2024
e2cf0e8
Fix format
chalin Aug 9, 2024
c782083
Fix capitalization
chalin Aug 9, 2024
9f70156
Fix link into spec, the hash doesn't exist
chalin Aug 9, 2024
e5151f3
Copyedit
chalin Aug 9, 2024
3f70a41
Fix typo
jack-berg Aug 12, 2024
6e75629
update opentelemetry-java-examples to latest
jack-berg Aug 12, 2024
e2e128e
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
jack-berg Sep 9, 2024
45b6b8a
merge main, fix build error
svrnm Sep 13, 2024
9cab82c
Merge branch 'main' of https://github.com/open-telemetry/opentelemetr…
jack-berg Sep 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .cspell/en-words.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,7 @@ jaeger
jaegertracing
javaagent
javadoc
javadocs
jboss
jdbc
julia
Expand Down
2 changes: 1 addition & 1 deletion .htmltest.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ IgnoreDirs:
- ^blog/(\d+/)?page/\d+
IgnoreInternalURLs: # list of paths
IgnoreURLs: # list of regexs of paths or URLs to be ignored
- ^(/..)?/docs/languages/\w+/(api|examples|registry)/$
- ^((/..)?/docs/languages/\w+|\.\.)/(api|examples|registry)/$
- ^(/..)?/docs/collector/registry/$
- ^(/..)?/docs/languages/net/(metrics-api|traces-api)/
- ^((/..)?/docs/migration/)?opencensus/$
Expand Down
1 change: 1 addition & 0 deletions .textlintrc.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ rules:
- '(?<![.-])java\b(?![.])'
- Java
- ['java[- ]?agent', Java agent]
- ['java ?doc(s)?', 'Javadoc$1']
Copy link
Contributor

Choose a reason for hiding this comment

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

@svrnm @theletterf - what do you think about this rule?

Copy link
Member

Choose a reason for hiding this comment

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

Would it catch "Java documentation" as well? So, if I write "to learn more about this, read the "Java documentation". Otherwise I am fine, since Javadoc seems to be the official notation

Copy link
Member

Choose a reason for hiding this comment

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

This may be an issue for sentences like "You will find more details on that in the Java documentation" meaning our internal java documentation. So we need to make sure that it does only capture "java doc(s)" with a word boundary (\b if I am not wrong?)

- [mac ?os, macOS]
- ['meta[- ]data', metadata]
- [nd-?json, NDJSON]
Expand Down
1 change: 1 addition & 0 deletions content/en/docs/languages/java/_index.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ cascade:
vers:
instrumentation: 2.8.0
otel: 1.42.1
contrib: 1.38.0
semconv: 1.27.0
weight: 18
---
Expand Down
982 changes: 627 additions & 355 deletions content/en/docs/languages/java/configuration.md
View file
Open in desktop

Large diffs are not rendered by default.

Loading