Skip to content

GitHub Actions workflow simplified#275

Merged
grunch merged 4 commits into
mainfrom
fix/gh-action-remove-ios
Aug 21, 2025
Merged

GitHub Actions workflow simplified#275
grunch merged 4 commits into
mainfrom
fix/gh-action-remove-ios

Conversation

@grunch

@grunch grunch commented Aug 21, 2025

Copy link
Copy Markdown
Member
  1. ✅ Updated Description: Changed from "Android and iOS" to "Android only"
  2. ✅ Changed Runner: macos-latest → ubuntu-latest (faster, cheaper, no iOS needed)
  3. ✅ Removed iOS Build: Eliminated "Build IPA" step entirely
  4. ✅ Removed iOS Compression: Eliminated "Compress Archives and IPAs" step
  5. ✅ Removed Alpha Logic: Eliminated both "Check if Tag Exists" and "Modify Tag" steps
  6. ✅ Cleaned Artifacts: Removed build/ios_build.tar.gz from uploads and releases
  7. ✅ Simplified Releases: Only APK and App Bundle files now

Benefits Achieved:

  • 🚫 No More iOS 18.0 Errors: Completely eliminated iOS build issues
  • ⚡ Faster Builds: Ubuntu runners are significantly faster than macOS
  • 💰 Cost Savings: Ubuntu runners are more cost-effective
  • 🎯 Simplified Logic: No complex alpha versioning edge cases
  • 📦 Cleaner Artifacts: Only Android builds (APK + AAB)
  • 🔧 Easier Maintenance: Streamlined workflow with fewer failure points

The GitHub Actions workflow is now focused solely on Android builds and will run much more
reliably without the persistent iOS SDK issues. When you're ready to add iOS back later, you can
do so once the platform availability issues in CI are resolved.

Summary by CodeRabbit

  • Chores
    • CI workflow simplified to run on Ubuntu for Android builds.
    • Removed iOS build, packaging, and upload steps; release artifacts now Android-only (APK/AAB).
    • Removed automated tag creation/modification from the release process.
    • Release creation adjusted to publish Android artifacts with options to replace or update without altering release body.

  1. ✅ Updated Description: Changed from "Android and iOS" to "Android only"
  2. ✅ Changed Runner: macos-latest → ubuntu-latest (faster, cheaper, no iOS needed)
  3. ✅ Removed iOS Build: Eliminated "Build IPA" step entirely
  4. ✅ Removed iOS Compression: Eliminated "Compress Archives and IPAs" step
  5. ✅ Removed Alpha Logic: Eliminated both "Check if Tag Exists" and "Modify Tag" steps
  6. ✅ Cleaned Artifacts: Removed build/ios_build.tar.gz from uploads and releases
  7. ✅ Simplified Releases: Only APK and App Bundle files now

  Benefits Achieved:

  - 🚫 No More iOS 18.0 Errors: Completely eliminated iOS build issues
  - ⚡ Faster Builds: Ubuntu runners are significantly faster than macOS
  - 💰 Cost Savings: Ubuntu runners are more cost-effective
  - 🎯 Simplified Logic: No complex alpha versioning edge cases
  - 📦 Cleaner Artifacts: Only Android builds (APK + AAB)
  - 🔧 Easier Maintenance: Streamlined workflow with fewer failure points

  The GitHub Actions workflow is now focused solely on Android builds and will run much more
  reliably without the persistent iOS SDK issues. When you're ready to add iOS back later, you can
  do so once the platform availability issues in CI are resolved.
@coderabbitai

coderabbitai Bot commented Aug 21, 2025

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@grunch has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 0 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 361de26 and 449f1cf.

📒 Files selected for processing (1)
  • .github/workflows/main.yml (3 hunks)

Walkthrough

The GitHub Actions workflow was switched to Ubuntu and restricted to Android: iOS build/packaging and tag-management steps were removed; artifact upload/release now only publishes Android APK/AAB and release creation uses replace/allow-update/omit-body options.

Changes

Cohort / File(s) Summary of edits
Workflow platform and scope
.github/workflows/main.yml
Runner changed from macos-latest to ubuntu-latest; workflow scoped to Android-only.
iOS build & packaging removed
.github/workflows/main.yml
Removed iOS build steps (Build IPA, Compress Archives/IPAs) and all references to ios_build.tar.gz.
Artifact upload & release simplified
.github/workflows/main.yml
Upload step no longer uploads ios_build.tar.gz; Create Release updated to publish only APK/AAB and set replace/artifact-update/omit-body options.
Tag management removal
.github/workflows/main.yml
Deleted steps for checking and modifying Git tags; no dynamic tag creation/modification logic remains.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant GH as GitHub
  participant WF as Actions Runner (ubuntu-latest)
  participant Release as GitHub Release

  Dev->>GH: Push / create release trigger
  GH->>WF: Start Android-only workflow
  rect rgba(200,230,255,0.25)
    note right of WF: Android build phase
    WF->>WF: Assemble APK
    WF->>WF: Build AAB
  end
  WF->>GH: Upload Android artifacts (APK, AAB)
  rect rgba(220,255,220,0.25)
    note right of WF: Release creation
    WF->>Release: Create/Update release (replace artifacts, allow updates, omit body)
  end
  note over WF,Release: iOS packaging and tag-management steps removed
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I hop through YAML fields so bright,
Swapped macOS for Ubuntu light—
Trimmed the iOS crumbs away,
APKs and AABs lead the day.
No tag-chases, just builds that hum—
Thump-thump: release is done. 🐇📦

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/gh-action-remove-ios

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

  • Visit our Status Page to check the current availability of CodeRabbit.
  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 3

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

💡 Knowledge Base configuration:

  • MCP integration is disabled by default for public repositories
  • Jira integration is disabled by default for public repositories
  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between a77896c and 361de26.

📒 Files selected for processing (1)
  • .github/workflows/main.yml (3 hunks)
🧰 Additional context used
🪛 actionlint (1.7.7)
.github/workflows/main.yml

3-3: unexpected key "description" for "workflow" section. expected one of "concurrency", "defaults", "env", "jobs", "name", "on", "permissions", "run-name"

(syntax-check)

🪛 YAMLlint (1.37.1)
.github/workflows/main.yml

[error] 14-14: trailing spaces

(trailing-spaces)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build

Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/main.yml Outdated
grunch and others added 3 commits August 21, 2025 10:28
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
...failures on reruns

On rerunning the workflow (e.g. via workflow_dispatch) with the same v${{ env.VERSION }} tag, the release-action will error out if the tag already exists. To make the step idempotent and update the existing release instead of failing, enable the following inputs (all supported by ncipollo/release-action@v1(github.com)):
@grunch grunch merged commit 5e276ab into main Aug 21, 2025
1 check passed
@grunch grunch deleted the fix/gh-action-remove-ios branch August 21, 2025 13:35
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