Skip to content

Conversation

heckj
Copy link
Contributor

@heckj heckj commented Aug 21, 2025

  • added explicit curation for the major types
  • explicitly hides some of the types that aren't meant to be public
  • revised the language to writing guidelines
    • removed symbols in abstracts
    • spelled out symbols in abstracts
  • revised titles to single sentences
  • updated grammar to switch to present tense

(the @_documentation updates reflect as breaking API changes, even though the availability of the symbols themselves aren't changed)

@heckj heckj changed the title doc formats metrics updates doc updates - formatting, organization, and language Aug 21, 2025
@heckj
Copy link
Contributor Author

heckj commented Aug 21, 2025

Checking the sound-ness manually:

swift package add-dependency https://github.com/swiftlang/swift-docc-plugin --from 1.0.0
swift package plugin generate-documentation --target "Metrics" --warnings-as-errors --analyze
swift package plugin generate-documentation --target "CoreMetrics" --warnings-as-errors --analyze
swift package plugin generate-documentation --target "MetricsTestKit" --warnings-as-errors --analyze

On Swift 6.2, all are passing without issue, but something is crashing in the GitHub workflow test pipeline for docs soundness.
The same works fine for checking against Swift 6.1, but the fails on Swift 6.0, which uses different(now legacy) disambiguation patterns for symbols and doesn't understand 6.1 and newer disambiguations.

@ktoso
Copy link
Member

ktoso commented Aug 25, 2025

Thanks Joe! The api breakage isn’t real because it’s about underscored symbols and we’re just hiding them from docs.

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.

Looks good! Just some minor notes

@ktoso ktoso added the semver/none No version bump required. label Aug 25, 2025
heckj and others added 10 commits August 26, 2025 09:05
- added explicit curation for the major types
- revised the language to writing guidelines
  - removed symbols in abstracts
  - spelled out symbols in abstracts
- revised titles to single sentences
- updated grammar to switch to present tense
Co-authored-by: Konrad `ktoso` Malawski <konrad.malawski@project13.pl>
@heckj
Copy link
Contributor Author

heckj commented Aug 26, 2025

I didn't add in the docs for the no-op handler, which depresses the coverage numbers a smidge, but most of the rest are in good shape in terms of just broad coverage and consistency:

                | Abstract        | Curated         | Code Listing
Types           | 100% (16/16)    | 100% (16/16)    | 38% (6/16)
Members         | 82% (113/137)   | 93% (127/137)   | 0.0% (0/137)
Globals         | 50% (1/2)       | 100% (2/2)      | 50% (1/2)

@heckj
Copy link
Contributor Author

heckj commented Sep 2, 2025

@ktoso if you're good with this, I'll need you to merge - I don't have commit access to override the semver issue that hiding the symbols caused.

@ktoso ktoso enabled auto-merge (squash) September 5, 2025 00:31
@ktoso ktoso disabled auto-merge September 5, 2025 00:32
@ktoso
Copy link
Member

ktoso commented Sep 5, 2025

The API breaks are not real:


  💔 API breakage: var Counter._handler is now with @_documentation
  💔 API breakage: var Counter._factory is now with @_documentation
  💔 API breakage: var FloatingPointCounter._handler is now with @_documentation
  💔 API breakage: var FloatingPointCounter._factory is now with @_documentation
  💔 API breakage: var Meter._handler is now with @_documentation
  💔 API breakage: var Meter._factory is now with @_documentation
  💔 API breakage: var Recorder._handler is now with @_documentation
  💔 API breakage: var Recorder._factory is now with @_documentation
  💔 API breakage: var Timer._handler is now with @_documentation
  💔 API breakage: var Timer._factory is now with @_documentation

merging, thank you!

@ktoso ktoso merged commit 0743a93 into apple:main Sep 5, 2025
31 of 32 checks passed
@heckj heckj deleted the docs-updates branch September 19, 2025 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver/none No version bump required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants