Skip to content

Conversation

heckj
Copy link
Contributor

@heckj heckj commented Sep 24, 2025

Documentation pass for swift distributed tracing

  • adds curation (organization) for all the symbols
    • shifts any deprecated methods or types to the bottom of lists
  • adds abstracts for all public types
  • light copy editing to apply standard Swift styling

@ktoso ktoso added semver/none No version bump required. area/documentation Improvements or additions to documentation. labels Sep 25, 2025
Copy link
Member

@ktoso ktoso left a comment

Choose a reason for hiding this comment

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

A few comments for now

@heckj
Copy link
Contributor Author

heckj commented Sep 25, 2025

Coverage isn't perfect, but significantly improved:

Instrumentation:

   --- Experimental coverage output enabled. ---
                | Abstract        | Curated         | Code Listing
Types           | 100% (8/8)      | 75% (6/8)       | 0.0% (0/8)
Members         | 92% (11/12)     | 17% (2/12)      | 0.0% (0/12)
Globals         | 100% (1/1)      | 100% (1/1)      | 0.0% (0/1)

InMemoryTracing:

   --- Experimental coverage output enabled. ---
                | Abstract        | Curated         | Code Listing
Types           | 89% (8/9)       | 44% (4/9)       | 0.0% (0/9)
Members         | 71% (67/94)     | 31% (29/94)     | 0.0% (0/94)
Globals         | 33% (1/3)       | 100% (3/3)      | 0.0% (0/3)

Tracing:

   --- Experimental coverage output enabled. ---
                | Abstract        | Curated         | Code Listing
Types           | 74% (25/34)     | 71% (24/34)     | 0.0% (0/34)
Members         | 83% (141/169)   | 83% (141/169)   | 0.0% (0/169)
Globals         | 30% (10/33)     | 100% (33/33)    | 0.0% (0/33)

@heckj heckj marked this pull request as ready for review September 25, 2025 22:25
Co-authored-by: Konrad `ktoso` Malawski <[email protected]>
@heckj heckj requested a review from ktoso September 29, 2025 13:37
@ktoso
Copy link
Member

ktoso commented Sep 30, 2025

Thank you, Joe! Awesome updates :-)

@ktoso ktoso merged commit d9c3031 into apple:main Oct 1, 2025
38 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Improvements or additions to documentation. semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants