Skip to content

CI build for iOS app#83

Closed
tillahoffmann wants to merge 2 commits intoonnela-lab:mainfrom
tillahoffmann:build
Closed

CI build for iOS app#83
tillahoffmann wants to merge 2 commits intoonnela-lab:mainfrom
tillahoffmann:build

Conversation

@tillahoffmann
Copy link

@tillahoffmann tillahoffmann commented Feb 5, 2026

@biblicabeebli, following on from our discussion yesterday, I'm going to set up a bit of testing infrastructure around the iOS app so I don't break anything in the process of exploring our new feature.

The main changes are

  • use rbenv to isolate ruby environment for reproducible builds
  • work around the bug in the unmaintained library using a post-install hook
  • add GitHub Action to build the app

tillahoffmann and others added 2 commits February 4, 2026 15:19
- Add Gemfile pinning CocoaPods ~> 1.16.2 and .ruby-version (3.3.10) for
  reproducible dependency management via rbenv/Bundler
- Fix PKHUD build error by setting IS_FRAMEWORK_TARGET compilation condition
  in Podfile post_install hook instead of manual source edits
- Add GitHub Actions workflow to build on main, PRs to main, and
  tillahoffmann/* branches
- Add vendor/ to .gitignore
- Update README with new build instructions

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
if: >-
github.event_name == 'pull_request'
|| github.ref == 'refs/heads/main'
|| github.repository_owner == 'tillahoffmann'
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Temporary injection to build on my fork so I don't have to create a PR for every change to get CI to run.

@tillahoffmann
Copy link
Author

Superseded by #84 (same branch pushed to upstream to enable CI)

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.

1 participant