Skip to content

Ignore ditto zip failures in iOS test action#99

Open
ssestak wants to merge 3 commits intomainfrom
fix/ios-test-ditto-error
Open

Ignore ditto zip failures in iOS test action#99
ssestak wants to merge 3 commits intomainfrom
fix/ios-test-ditto-error

Conversation

@ssestak
Copy link
Copy Markdown
Collaborator

@ssestak ssestak commented Apr 9, 2026

Summary

  • Fixes ditto: Can't archive multiple sources error that fails iOS test jobs (e.g. gmlh-ios)
  • Adds || true to the ditto zip command so failures are silently ignored
  • Conditionally skips the upload step if no zip was produced
  • Note: continue-on-error does not work in composite action steps — shell-level suppression is required

Important

After merging, a new version tag must be created for consumer repos to pick up the fix (all action references are pinned to @2.3.0).

Test plan

  • Verify iOS test workflow still passes when .app is present (zip + upload work as before)
  • Verify iOS test workflow passes when no .app is found (ditto fails silently, job succeeds)
  • Verify iOS test workflow passes when multiple .app files exist (the gmlh-ios case)

Šimon Šesták added 2 commits April 9, 2026 17:05
The ditto command fails when no .app is found in DerivedData,
which is non-critical and should not fail the test job.
…bility

continue-on-error is not supported in composite action steps.
Use shell-level error suppression and conditional upload instead.
@ssestak ssestak force-pushed the fix/ios-test-ditto-error branch from bd454b4 to 59a145a Compare April 9, 2026 17:19
@ssestak ssestak marked this pull request as ready for review April 9, 2026 17:21
@ssestak ssestak force-pushed the fix/ios-test-ditto-error branch from 9935f68 to ded0d11 Compare April 9, 2026 17:26
@ssestak ssestak requested a review from jmarek41 April 9, 2026 17:26
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.

2 participants