Skip to content

Commit dad4229

Browse files
authored
Update README for coverage report and warnings clarity
Clarified coverage report export and GitHub Actions warnings handling in README.
1 parent c5a4b59 commit dad4229

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -444,7 +444,7 @@ The workflow currently runs a single job named `test-and-analyze` on `macos-late
444444
- enables code coverage during test execution
445445
- writes the test result bundle to `TestResults.xcresult`
446446
- runs `xcodebuild analyze` for static analysis after tests
447-
- exports a human-readable coverage report with `xcrun xccov view --report`
447+
- exports a coverage report with `xcrun xccov view --report`
448448
- uploads the generated `coverage.txt` file as a GitHub Actions artifact named `weatherapp-coverage`
449449

450450
In practical terms, CI is enforcing three things on every push and pull request:
@@ -453,11 +453,11 @@ In practical terms, CI is enforcing three things on every push and pull request:
453453
- the unit tests must pass on the configured simulator destination
454454
- Xcode static analysis must complete without failing the workflow
455455

456-
The artifact handling is intentionally simple. Coverage is not yet posted as a PR comment, uploaded to a third-party dashboard, or used as a merge gate. Instead, the workflow keeps the `coverage.txt` file as a downloadable build artifact for manual inspection from the Actions run page.
456+
The artifact handling is intentionally simple. Coverage is not yet posted as a PR comment. Instead, the workflow keeps the `coverage.txt` file as a downloadable build artifact for manual inspection from the Actions run page.
457457

458458
There is no CD step yet. The repository does not currently build release archives, sign the app, upload to TestFlight, deploy metadata, or publish artifacts beyond the test coverage report. If release automation is added later, it should be kept separate from the validation job so CI failures and release failures remain easy to distinguish.
459459

460-
If GitHub Actions shows JavaScript runtime warnings such as Node 20 deprecation notices, those warnings are about the GitHub-hosted action runtime used by actions like `actions/checkout`, not about the Swift app itself. They should be handled by updating workflow action versions when newer compatible releases are available.
460+
GitHub Actions currently shows JavaScript runtime warnings such as Node 20 deprecation notices, those warnings are about the GitHub-hosted action runtime used by actions like `actions/checkout`, not about the Swift app itself. They should be handled by updating workflow action versions when newer compatible releases are available.
461461

462462
## Static Analysis
463463

0 commit comments

Comments
 (0)