Skip to content

Fix Swift activity summaries to include positive steps - #1477

Merged
ryanbr merged 1 commit into
ryanbr:mainfrom
bhelm:fix/activity-summary-steps
Aug 20, 2026
Merged

ryanbr merged 1 commit into
ryanbr:mainfrom
bhelm:fix/activity-summary-steps

Conversation

@bhelm

@bhelm bhelm commented Aug 20, 2026

Copy link
Copy Markdown

Summary

  • append positive step counts to Swift activity import summaries, matching Kotlin
  • preserve existing output for nil and zero steps
  • add mirrored Swift and Kotlin parity tests

Validation

  • Swift focused: 1 passed
  • Swift affected ActivityFileImporterTests: 11 passed
  • Swift full StrandImport: 233 passed, 0 failed, 1 environment-dependent skip
  • Kotlin focused and affected: passed
  • Kotlin FullDebug: 4092 passed, 0 failed, 0 errors, 6 skipped
  • independent final delta review: PASS, no P0/P1/P2

Fixes bhelm#91.

@ryanbr
ryanbr merged commit c821c40 into ryanbr:main Aug 20, 2026
14 checks passed
@bhelm
bhelm deleted the fix/activity-summary-steps branch September 15, 2026 22:35
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.

[P2] Activity import completion summary omits positive steps on Swift

2 participants