Page MenuHomePhabricator

[native] Bump `react-native-svg` to `12.3.0`
ClosedPublic

Authored by atul on Apr 27 2022, 7:35 AM.
Tags
None
Referenced Files
F3380373: D3856.diff
Wed, Nov 27, 11:09 PM
Unknown Object (File)
Sun, Nov 24, 7:52 AM
Unknown Object (File)
Sun, Nov 24, 7:38 AM
Unknown Object (File)
Sun, Nov 24, 7:36 AM
Unknown Object (File)
Sun, Nov 24, 7:25 AM
Unknown Object (File)
Fri, Nov 8, 2:56 AM
Unknown Object (File)
Fri, Nov 8, 2:56 AM
Unknown Object (File)
Fri, Nov 8, 2:56 AM

Details

Summary

Downstream dependencies have issues surfaced by yarn audit

Checked the release notes: https://github.com/react-native-svg/react-native-svg/releases and didn't find any breaking changes


Depends on D3855

Test Plan

CI

Diff Detail

Repository
rCOMM Comm
Branch
landapr27 (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul requested review of this revision.Apr 27 2022, 7:40 AM
  1. Please make sure to make me a blocking reviewer on diffs that affect dependencies in the future
  2. Can you also test wherever we use react-native-svg to make sure it still works as expected?
This revision is now accepted and ready to land.Apr 27 2022, 2:30 PM

Can you also test wherever we use react-native-svg to make sure it still works as expected?

Yeah, we only use it to display the "background" tab illustration. I'll check it before landing.

cherrypick before testing before landing

include changes to Podfile.lock

Good thing I didn't land it before testing the "background illustration" SVG.

I'm on my old MacBook Air and it looks like my ruby environment set up was out-of-date. As a result pod install failed silently and so the Podfile.lock file never updated.

Fixed the ruby environment, ensured that pod install ran correctly, ensured that Podfile.lock changed as expected, and made sure that the "background illustration" SVG looks as expected on iOS:

Simulator Screen Shot - iPhone 13 mini - 2022-04-28 at 13.26.21.png (2×1 px, 213 KB)

This revision was landed with ongoing or failed builds.Apr 28 2022, 11:28 AM
This revision was automatically updated to reflect the committed changes.