Skip to content

Fix: skip build job when demo matrix is empty#3736

Merged
cpholguera merged 2 commits intoOWASP:masterfrom
Diolor:fix/build-ios-demos-empty-matrix
Mar 5, 2026
Merged

Fix: skip build job when demo matrix is empty#3736
cpholguera merged 2 commits intoOWASP:masterfrom
Diolor:fix/build-ios-demos-empty-matrix

Conversation

@Diolor
Copy link
Collaborator

@Diolor Diolor commented Mar 5, 2026

Description

When a PR has changed demos/android/ but no changed files in demos/ios/, the generate-matrix job outputs an empty array which causes GitHub Actions to error with "Matrix vector 'demo' does not contain any values". Adding an if condition skips the build job entirely in that case.

See the error here : https://github.com/OWASP/mastg/actions/runs/22710797554

Vice versa for Android flow.


AI Tool Disclosure

  • [ x] This contribution includes AI-generated content.

Diolor and others added 2 commits March 5, 2026 10:26
When a PR has no changed files in demos/ios/, the generate-matrix job
outputs an empty array which causes GitHub Actions to error with
"Matrix vector 'demo' does not contain any values". Adding an `if`
condition skips the build job entirely in that case.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Same issue as the iOS workflow: when a PR has no changed files in
demos/android/, the empty matrix causes "Matrix vector 'demo' does not
contain any values". Adding an `if` condition skips the build job.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@Diolor Diolor self-assigned this Mar 5, 2026
@cpholguera cpholguera merged commit a551f11 into OWASP:master Mar 5, 2026
12 checks passed
@Diolor Diolor deleted the fix/build-ios-demos-empty-matrix branch March 5, 2026 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants