Skip to content

Remove unused react-native-reanimated from the sample - #537

Merged
kieran-osgood-shopify merged 1 commit into
dependency-sweep/androidfrom
dependency-sweep/remove-reanimated
Oct 2, 2026
Merged

kieran-osgood-shopify merged 1 commit into
dependency-sweep/androidfrom
dependency-sweep/remove-reanimated

Conversation

@kieran-osgood-shopify

@kieran-osgood-shopify kieran-osgood-shopify commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor

What changes are you making?

Removes react-native-reanimated from the sample. Nothing uses it.
Affects sample only


PR Checklist

Important

Releasing a new version of the kit?


Tip

See the Contributing documentation for instructions on how to publish a new version of the library.

@kieran-osgood-shopify
kieran-osgood-shopify added this pull request to stack #538 October 1, 2026 10:53
@github-actions

github-actions Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Coverage Report

Lines Statements Branches Functions
Coverage: 99%
99.12% (227/229) 94.61% (123/130) 100% (67/67)

@kieran-osgood-shopify
kieran-osgood-shopify marked this pull request as ready for review October 1, 2026 12:26
@kieran-osgood-shopify
kieran-osgood-shopify requested a review from a team as a code owner October 1, 2026 12:26
Nothing in the sample, module, or scripts imports Reanimated, and no
installed dependency requires it, yet it was autolinked into both native
apps. Drop the dependency, its autolinking entry, the stale Xcode header
search paths, and the RNReanimated pods.
@kieran-osgood-shopify
kieran-osgood-shopify force-pushed the dependency-sweep/remove-reanimated branch from 27eaa93 to 96f9b01 Compare October 2, 2026 09:30
@kieran-osgood-shopify
kieran-osgood-shopify merged commit 9b42023 into main Oct 2, 2026
15 checks passed
@kieran-osgood-shopify
kieran-osgood-shopify deleted the dependency-sweep/remove-reanimated branch October 2, 2026 09:46
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.

2 participants