Skip to content

Conversation

@transphorm
Copy link
Member

@transphorm transphorm commented Oct 2, 2025

Summary

  • restore the iOS and Android build bump pull request steps in the mobile deploy workflow
  • remove the pull request preview step so build bump automation runs exactly as before
  • ensure the version update automation skips pushing branches when running inside pull request workflows

Testing

  • not run (workflow change only)

https://chatgpt.com/codex/tasks/task_b_68ddfd5106f8832dbf26ea5962a108ee

Summary by CodeRabbit

  • Chores
    • Automated creation of pull requests to bump iOS and Android build numbers after successful workflow runs.
    • Added gating to ensure automation runs only in appropriate environments and events.
    • Implemented a dry-run mode on pull request events to preview version changes without committing or modifying repository state.
    • Streamlines release preparation by centralizing version updates into dedicated PRs, improving visibility and reducing manual steps.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 2, 2025

Warning

Rate limit exceeded

@transphorm has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 4 minutes and 5 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 14cfbb0 and b842027.

📒 Files selected for processing (2)
  • .github/workflows/mobile-deploy.yml (3 hunks)
  • app/version.json (1 hunks)

Walkthrough

Adds automated PR creation steps in mobile-deploy workflow to bump iOS and Android build numbers using peter-evans/create-pull-request, includes app/version.json in PRs, and introduces a pull_request dry-run path that outputs diffs and exits without committing or opening PRs.

Changes

Cohort / File(s) Summary of Changes
iOS build-number PR creation
.github/workflows/mobile-deploy.yml
Adds step to open a PR to staging for iOS build number bump via peter-evans/create-pull-request@v6 with commit message, title, body, branch, and inclusion of app/version.json; gated to skip on ACT and pull_request events and only run on success.
Android build-number PR creation
.github/workflows/mobile-deploy.yml
Adds analogous step for Android build number bump PR creation with identical gating and parameters.
Shared version-update flow and dry-run
.github/workflows/mobile-deploy.yml
Adds shared PR creation after version retrieval; introduces guard for pull_request events to print staged diffs, avoid commits, and exit early.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor Dev as Developer
  participant GH as GitHub Actions Runner
  participant WF as mobile-deploy.yml
  participant PE as peter-evans/create-pull-request

  Dev->>GH: Push / Dispatch workflow
  GH->>WF: Run update-version steps
  alt Event == pull_request
    WF->>WF: Generate diffs (dry-run)
    WF-->>GH: Output staged changes and exit
  else Not pull_request and not ACT
    WF->>WF: Apply version/build bumps
    par iOS path
      WF->>PE: Create PR for iOS build bump (into staging)
      PE-->>WF: PR URL/metadata
    and Android path
      WF->>PE: Create PR for Android build bump (into staging)
      PE-->>WF: PR URL/metadata
    end
  end
Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • remicolin
  • aaronmgdr

Poem

Bumps on the runway, versions take flight,
iOS and Android queued just right.
Dry-run whispers, “no commit tonight,”
PRs assemble in staging light.
YAML hums—an automated rite. ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title succinctly describes the core change of the workflow update by specifying the restoration of the mobile build bump PR automation, making it clear and relevant to the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 49b0e55 and 14cfbb0.

📒 Files selected for processing (1)
  • .github/workflows/mobile-deploy.yml (3 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
.github/workflows/**/*.{yml,yaml}

📄 CodeRabbit inference engine (AGENTS.md)

.github/workflows/**/*.{yml,yaml}: In GitHub Actions workflows, use shared composite caching actions from .github/actions (cache-yarn, cache-bundler, cache-gradle, cache-pods)
Do not call actions/cache directly; rely on the shared composite caching actions
When using cache actions, optionally pass cache-version (often with GH_CACHE_VERSION and tool version) for stable keys

Files:

  • .github/workflows/mobile-deploy.yml

@transphorm transphorm merged commit 6d935c4 into dev Oct 2, 2025
22 checks passed
@transphorm transphorm deleted the codex/restore-pr-logic-for-mobile-app-deploy branch October 2, 2025 14:42
@coderabbitai coderabbitai bot mentioned this pull request Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants