Skip to content

Conversation

@mabdinur
Copy link
Contributor

@mabdinur mabdinur commented Dec 31, 2025

What does this PR do? What is the motivation?

Restructures OpenTelemetry API support documentation to follow a consistent multi-language directory structure:

  • Rust: Creates rust/ directory with metrics.md (private, v0.4.0) and traces.md, removes old rust.md
  • Go: Creates go/ directory with metrics.md and traces.md, following the same directory-based organization
  • PHP: Creates php/ directory with metrics.md and traces.md, following the same directory-based organization

Structure:

  • Rust, Go, and PHP now follow the same directory-based organization as Python, .NET, Node.js, and Ruby

Motivation:

  • Standardize documentation structure across all supported languages and prepare for upcoming metrics support in Rust (v0.3.0), Go (v2.6.0), and PHP (v1.16.0).

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

  • Rust metrics tab is hidden (private: true) until v0.4.0 release
  • Documentation based strictly on existing dd-trace-rs implementation
  • No new content added, only restructuring and formatting
  • Go and PHP restructuring will follow in separate commits

@github-actions
Copy link
Contributor

github-actions bot commented Dec 31, 2025

Preview links (active after the build_preview check completes)

New or renamed files

Removed or renamed files (these should redirect)

@mabdinur mabdinur force-pushed the munir/opentelemetry-metrics-docs-2 branch from 79bca67 to 5e304fd Compare January 2, 2026 17:20
@mabdinur mabdinur requested a review from rachelyangdog January 5, 2026 22:13
@mabdinur mabdinur marked this pull request as ready for review January 16, 2026 20:12
@mabdinur mabdinur requested a review from a team as a code owner January 16, 2026 20:12
@mabdinur mabdinur requested a review from brett0000FF January 16, 2026 20:12
@OliviaShoup OliviaShoup added the editorial review Waiting on a more in-depth review label Jan 16, 2026
@OliviaShoup
Copy link
Contributor

thank you for the PR! just a heads up: because of the size, it'll take a little longer to be reviewed by the docs team

@mabdinur
Copy link
Contributor Author

thank you for the PR! just a heads up: because of the size, it'll take a little longer to be reviewed by the docs team

No rush on this. This PR depends on #33640, either one can be merged first, but merging one will introduce conflicts that will need to be resolved in the other.

@brett0000FF
Copy link
Contributor

@mabdinur - Since we merged that api_support update that restructures these pages, I took a crack at folding your new content into that structure. Let me know how this looks! #34129

@mabdinur
Copy link
Contributor Author

Closing in favor of: #34129

@mabdinur mabdinur closed this Jan 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

editorial review Waiting on a more in-depth review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants