Skip to content

Conversation

@heckj
Copy link
Member

@heckj heckj commented Apr 15, 2025

  • verified with Swift 5.10, 6, and 6.1. Prior to swift 6, level was related to providing analysis coverage, but that doesn't appear to be the intent with this check.
  • with Swift 5.10, the --level option was ignored - no warning was generated
  • post Swift 6, that option was renamed --coverage-summary-level, but still is a no-op without also including
    the option --experimental-documentation-coverage

resolves #116

- verified with Swift 5.10, 6, and 6.1. Prior to swift 6, level was related to providing analysis coverage, but that doesn't appear to be the intent with this check.
- with Swift 5.10, the --level option was ignored - no warning was generated
- post Swift 6, that option was renamed --coverage-summary-level, but still is a no-op without also including
the option --experimental-documentation-coverage
@heckj heckj requested a review from a team as a code owner April 15, 2025 20:51
@shahmishal shahmishal merged commit 152921c into swiftlang:main Apr 15, 2025
25 of 26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

soundness -> documentation-check uses a deprecated option

2 participants