8000 fix(ios): record the integration_test pod the sensor lane added by shroominic · Pull Request #180 · shroominic/OpenGlucose · GitHub
[go: up one dir, main page]

Skip to content

fix(ios): record the integration_test pod the sensor lane added - #180

Closed
shroominic wants to merge 1 commit into
release/v0.4.0from
fix/ios-podfile-lock-integration-test
Closed

shroominic wants to merge 1 commit into
release/v0.4.0from
fix/ios-podfile-lock-integration-test

Conversation

@shroominic
Copy link
Copy Markdown
Owner

Unblocks the iOS gate on the assembled 0.4.0 candidate (#163).

Defect

The iOS job fails closed on lockfile drift:

error: the iOS build changed ios/Podfile.lock; update and review the lockfile explicitly

The sensor lane added the app's integration_test dev dependency and the openhealth/integration_test/ harnesses, so CocoaPods now resolves an integration_test pod the committed lockfile never recorded. main is unaffected: its ios/Podfile.lock blob (bd2e8e4) is identical to this branch's pre-image.

Change

Only the four additions the pinned toolchain produced on CI (Xcode 26.6, CocoaPods 1.16.2): the PODS, DEPENDENCIES, EXTERNAL SOURCES and SPEC CHECKSUMS entries for integration_test. No checksum, version, or dependency beyond that entry moves.

Verification

This host has no Xcode or CocoaPods (CommandLineTools only), so the diff was applied from the iOS job log and pinned by hash: git hash-object openhealth/ios/Podfile.lock is ad15bd2, byte-identical to the post-image the CI job generated (index bd2e8e4..ad15bd2). The iOS job on the aggregate is the run-level confirmation.

The sensor lane added the app's `integration_test` dev dependency and its
harness directory, so `pod install` now resolves an `integration_test` pod that
the committed lockfile never recorded. The iOS job fails closed on that drift:

  the iOS build changed ios/Podfile.lock; update and review the lockfile explicitly

This records the change the pinned toolchain produced on CI (Xcode 26.6,
CocoaPods 1.16.2): the PODS, DEPENDENCIES, EXTERNAL SOURCES and SPEC CHECKSUMS
entries for `integration_test` only. main is unaffected, its lockfile is
byte-identical to the pre-image.

No local Xcode or CocoaPods on this host, so the change is applied from the CI
diff and verified by blob hash: the result is ad15bd2, byte-identical to the
post-image the iOS job generated.
@shroominic
Copy link
Copy Markdown
Owner Author

Fixes #190.

@shroominic
Copy link
Copy Markdown
Owner Author

Carried by the GS1 consolidation in PR #209 (branch feature/cbio-gs1, head 3eca4f4).

This PR is contained in the consolidated line; the merge that carries it is c85f827.

Closing as subsumed by the consolidation - not as won't fix. The per-defect evidence stays in this thread, and the carrying commit above is where the change now lives.

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.

1 participant

0