mobile: bump required Ruby to 3.2.11, add rv note for Apple Silicon - #1518
Merged
Conversation
Bumps the required Ruby version to match .ruby-version (3.2.11) and documents rv as an alternative to RVM, which hits repeated OpenSSL errors on recent Apple Silicon Macs. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Newest code from mattermost has been published to preview environment for Git SHA 74066d7 |
|
Newest code from mattermost has been published to preview environment for Git SHA 74066d7 |
Contributor
📝 WalkthroughWalkthroughThe mobile developer setup documentation now uses the repository-pinned Ruby version, updates installation commands, and adds Apple Silicon troubleshooting guidance for RVM OpenSSL failures with an ChangesRuby setup documentation
Estimated code review effort: 1 (Trivial) | ~3 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@site/content/contribute/more-info/mobile/developer-setup/_index.md`:
- Around line 139-144: Update the Ruby setup instructions around the rv
installation commands to include the required shell integration step, using “rv
shell zsh” or the equivalent command for the user’s shell before the note about
automatic version switching. Keep the existing installation command and pinned
.ruby-version behavior unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 468c1c7d-944f-4370-8c12-16a333a43305
📒 Files selected for processing (1)
site/content/contribute/more-info/mobile/developer-setup/_index.md
rv only switches Ruby versions on cd after its shell hook is installed. Add the `eval "$(rv shell init zsh)"` setup step so the automatic switching claim holds. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Newest code from mattermost has been published to preview environment for Git SHA a464ab2 |
vish9812
added
the
Do Not Merge/Awaiting PR
2 tasks
enahum
approved these changes
Jul 30, 2026
vish9812
added
4: Reviews Complete
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates the mobile developer-setup guide to require Ruby 3.2.11 (matching
.ruby-versioninmattermost-mobile) and documentsrvas an alternative to RVM.Why
On recent Apple Silicon Macs, installing Ruby 3.2.0 via RVM hits repeated OpenSSL errors.
rvinstalls cleanly and auto-selects the pinned version from.ruby-version. The mobile repo has been bumped to 3.2.11 accordingly (mattermost/mattermost-mobile#9962).Changes
rvwhen RVM fails with OpenSSL errors.References
🤖 Generated with Claude Code