Skip to content

Add missing dependency in ios module and test iOS build in CI - #2841

Merged
riccardobl merged 7 commits into
masterfrom
ios
Jun 2, 2026
Merged

Add missing dependency in ios module and test iOS build in CI#2841
riccardobl merged 7 commits into
masterfrom
ios

Conversation

@riccardobl

@riccardobl riccardobl commented Jun 2, 2026

Copy link
Copy Markdown
Member

This PR adds the missing angle-ios dependency to jme3-ios and removes all explicit libjglios dependencies from jme3-ios-example, to test the build with transitive dependencies inherited from jme3-ios, as normal user project would do.

Added a new job to build iOS examples in the CI workflow.
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@riccardobl
riccardobl marked this pull request as ready for review June 2, 2026 06:45
@riccardobl riccardobl changed the title Add iOS build job to GitHub Actions workflow Test iOS build in CI Jun 2, 2026
@riccardobl riccardobl changed the title Test iOS build in CI Add missing dependency in ios module and test iOS build in CI Jun 2, 2026
@riccardobl
riccardobl merged commit 7b911d6 into master Jun 2, 2026
24 checks passed
@riccardobl
riccardobl deleted the ios branch June 8, 2026 07:58
Copilot AI pushed a commit that referenced this pull request Jul 15, 2026
* Add iOS build job to GitHub Actions workflow

Added a new job to build iOS examples in the CI workflow.

* Update Java version to 25 in workflow

* Update main.yml

* Rename iOS build step for clarity

* Update iOS build workflow to exclude debug app

Remove build command for iOS debug app in workflow

* Add angle library dependency for iOS build

* Remove unused iOS library dependencies from build.gradle

Removed unnecessary iOS library dependencies.
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