Skip to content

fix: remove package for ios built success#174

Merged
in-mai-space merged 4 commits intomainfrom
fix-fe-pod-install
Feb 27, 2026
Merged

fix: remove package for ios built success#174
in-mai-space merged 4 commits intomainfrom
fix-fe-pod-install

Conversation

@in-mai-space
Copy link
Contributor

@in-mai-space in-mai-space commented Feb 27, 2026

Description

How has this been tested?

Checklist

  • I have self-reviewed my code for readability, maintainability, performance, and added comments/documentation where necessary.
  • New and existing tests pass locally with my changes.
  • I have followed the project's coding standards and best practices.

User-visible changes

  • Replaced rn-phone-number-input library with libphonenumber-js to resolve iOS pod installation compatibility issues
  • Fixed iOS build process by removing conflicting native module dependency
  • Updated EAS configuration for improved build setup

Changes by category

Fixes

  • Removed outdated rn-phone-number-input (^0.3.2) dependency causing iOS pod install failures
  • Added libphonenumber-js (^1.12.38) as replacement phone number utility
  • Repaired eas.json configuration

Infra

  • Updated frontend package.json dependencies

Author contribution

Metric Count
Lines added 1
Lines removed 1
Net change 0

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1950125 and 02cf8fc.

⛔ Files ignored due to path filters (1)
  • frontend/bun.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • frontend/package.json

📝 Walkthrough

Walkthrough

The frontend package.json was updated to replace the phone number input library. The rn-phone-number-input dependency was removed and replaced with libphonenumber-js version 1.12.38.

Changes

Cohort / File(s) Summary
Dependency Update
frontend/package.json
Replaced rn-phone-number-input dependency with libphonenumber-js 1.12.38.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

Phone numbers need a home, it's true,
Old library out, new one shines through,
One line replaced with careful intent,
A lightweight swap, perfectly sent.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title does not accurately describe the main change. It says 'remove package for ios built success' but the actual change involves replacing one phone number library with another specific library (libphonenumber-js). Revise the title to be specific about the change, such as 'Replace rn-phone-number-input with libphonenumber-js for iOS compatibility' or similar wording that clearly indicates which package is being replaced and why.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-fe-pod-install

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.

@in-mai-space in-mai-space merged commit 5af1112 into main Feb 27, 2026
10 checks passed
@in-mai-space in-mai-space deleted the fix-fe-pod-install branch February 27, 2026 05:02
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