Skip to content

fix(react-native): Handle xcode build phase patching failure - #866

Merged
antonis merged 15 commits into
masterfrom
antonis/rn-handle-xcode-build-phase-failure
Apr 14, 2025
Merged

antonis merged 15 commits into
masterfrom
antonis/rn-handle-xcode-build-phase-failure

Conversation

@antonis

@antonis antonis commented Mar 18, 2025 •

Copy link
Copy Markdown
Contributor

Fixes #572

Handled Bundle React Native and images build phase patching gracefully and show a code snippet for manual configuration.

Screenshot 2025-03-19 at 2 42 32 PM

@codecov

codecov Bot commented Mar 18, 2025 •

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 80.26316% with 15 lines in your changes missing coverage. Please review.

Project coverage is 24.24%. Comparing base (9b082e9) to head (4c4c52b).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/react-native/xcode.ts 83.56% 12 Missing ⚠️
src/react-native/react-native-wizard.ts 0.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #866      +/-   ##
==========================================
+ Coverage   23.86%   24.24%   +0.38%     
==========================================
  Files         121      121              
  Lines       12753    12817      +64     
  Branches      701      711      +10     
==========================================
+ Hits         3043     3107      +64     
  Misses       9689     9689              
  Partials       21       21              
Flag Coverage Δ
unit-tests 24.24% <80.26%> (+0.38%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@antonis
antonis requested a review from krystofwoldrich March 18, 2025 18:14
@antonis
antonis marked this pull request as ready for review March 18, 2025 18:14
Comment thread src/react-native/xcode.ts Outdated
Comment thread src/react-native/xcode.ts
@antonis
antonis requested a review from krystofwoldrich March 19, 2025 12:45
@antonis
antonis marked this pull request as draft March 31, 2025 07:21
@antonis
antonis marked this pull request as ready for review March 31, 2025 07:28
Comment thread src/react-native/xcode.ts Outdated
antonis and others added 2 commits April 11, 2025 18:55
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
Comment thread src/react-native/xcode.ts Outdated
Comment thread src/react-native/xcode.ts
@krystofwoldrich

Copy link
Copy Markdown
Contributor

I've noticed two small visual details; otherwise, it looks great!

antonis and others added 3 commits April 14, 2025 11:09
Co-authored-by: Krystof Woldrich <31292499+krystofwoldrich@users.noreply.github.com>
@antonis
antonis requested a review from krystofwoldrich April 14, 2025 11:23

@krystofwoldrich krystofwoldrich 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.

LGTM 🚀 Thank you!

@antonis
antonis merged commit 4b5eec6 into master Apr 14, 2025
@antonis
antonis deleted the antonis/rn-handle-xcode-build-phase-failure branch April 14, 2025 14:50
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.

Sentry Wizard fails to modify Bundle React Native and images

2 participants