Skip to content

Revert "Add RNRepo to ci builds (#4141)"#4150

Closed
j-piasecki wants to merge 1 commit into
mainfrom
@jpiasecki/revert-rnrepo
Closed

Revert "Add RNRepo to ci builds (#4141)"#4150
j-piasecki wants to merge 1 commit into
mainfrom
@jpiasecki/revert-rnrepo

Conversation

@j-piasecki
Copy link
Copy Markdown
Member

This reverts commit 451eb50.

Description

expo-example on Android stopped working with

[Error: Exception in HostFunction: no non-static method "Lcom/swmansion/gesturehandler/react/RNGestureHandlerModule;.setReanimatedAvailable(Z)V"]

Code: NativeProxy.ts
  61 |   },
  62 |   setReanimatedAvailable: (isAvailable: boolean) => {
> 63 |     RNGestureHandlerModule.setReanimatedAvailable(isAvailable);
     |                                                  ^
  64 |   },
  65 | } as const;
  66 |
Call Stack
  setReanimatedAvailable (packages/react-native-gesture-handler/src/v3/NativeProxy.ts:63:50)
  <global> (packages/react-native-gesture-handler/src/handlers/gestures/reanimatedWrapper.ts:96:37)
  useAnimatedGesture (packages/react-native-gesture-handler/src/handlers/gestures/GestureDetector/useAnimatedGesture.ts:105:35)
  GestureDetector (packages/react-native-gesture-handler/src/handlers/gestures/GestureDetector/index.tsx:87:65)

Copilot AI review requested due to automatic review settings May 7, 2026 06:21
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Reverts the earlier RNRepo CI integration to restore expected native build behavior for the example apps (notably expo-example Android), by removing RNRepo tooling/config and undoing related CI and build-script changes.

Changes:

  • Removes RNRepo packages/configuration from the repo and example apps.
  • Reverts iOS/Android example-app build script modifications that enabled RNRepo in CI.
  • Adjusts CI workflows accordingly (including Android Gradle invocation and path filters).

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
yarn.lock Removes RNRepo and related Expo config-plugin lock entries introduced by the reverted change.
rnrepo.config.json Deletes RNRepo configuration file.
apps/expo-example/package.json Removes RNRepo Expo config plugin dev dependency.
apps/expo-example/app.config.js Removes RNRepo Expo config plugin from Expo plugins list.
apps/basic-example/package.json Removes RNRepo build-tools dev dependency.
apps/basic-example/ios/Podfile Removes RNRepo CocoaPods plugin enablement and post-install hook usage.
apps/basic-example/android/build.gradle Removes RNRepo Gradle plugin classpath wiring.
apps/basic-example/android/app/build.gradle Removes conditional RNRepo plugin application.
.github/workflows/ios-build.yml Updates PR path filters to drop RNRepo-related triggers.
.github/workflows/android-build.yml Updates PR path filters and changes the Gradle build command.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 6 to 8
- .github/workflows/android-build.yml
- packages/react-native-gesture-handler/android/**
- apps/basic-example/android/**
- name: Build app
working-directory: ${{ matrix.working-directory }}/android
run: ./gradlew :app:assembleDebug --console=plain -PreactNativeArchitectures=arm64-v8a
run: ./gradlew assembleDebug --console=plain -PreactNativeArchitectures=arm64-v8a
Comment on lines 3 to 10
on:
pull_request:
paths:
- .github/workflows/ios-build.yml
- packages/react-native-gesture-handler/RNGestureHandler.podspec
- packages/react-native-gesture-handler/apple/**
- apps/basic-example/ios/**
- apps/basic-example/package.json
- rnrepo.config.json
- yarn.lock
push:
@radoslawrolka
Copy link
Copy Markdown
Contributor

radoslawrolka commented May 7, 2026

#4151 was the fix @j-piasecki - this can be closed

@j-piasecki j-piasecki closed this May 7, 2026
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.

3 participants