Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: new arch measurements on Android #589

Merged
merged 3 commits into from
Apr 4, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions .github/workflows/ReactNativeSlider-CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ jobs:
device=$(xcrun simctl list devices "${device_name}" available | grep "${device_name} (")
re='\(([-0-9A-Fa-f]+)\)'
[[ $device =~ $re ]] || exit 1
xcodebuild -workspace example.xcworkspace -scheme example -destination "platform=iOS Simulator,id=${BASH_REMATCH[1]}" CODE_SIGNING_ALLOWED=NO COMPILER_INDEX_STORE_ENABLE=NO build
set -o pipefail && xcodebuild -workspace example.xcworkspace -scheme example -destination "platform=iOS Simulator,id=${BASH_REMATCH[1]}" CODE_SIGNING_ALLOWED=NO COMPILER_INDEX_STORE_ENABLE=NO build | xcbeautify --renderer github-actions
working-directory: example/ios


Expand Down Expand Up @@ -191,6 +191,9 @@ jobs:
path: example/ios/Pods
key: new-arch-${{ hashFiles('./example/ios/Podfile.lock') }}

- name: Use the current package sources in build
run: cd example && npm run refresh-package

- name: Install required dependencies on cache miss (Pods)
if: steps.cache-pods.outputs.cache-hit != 'true'
run: |
Expand All @@ -200,16 +203,13 @@ jobs:
if: steps.cache-pods.outputs.cache-hit == 'true'
run: cd example/ios && RCT_NEW_ARCH_ENABLED=1 pod install

- name: Use the current package sources in build
run: cd example && npm run refresh-package

- name: Build iOS - Fabric
run: |
device_name='iPhone 13'
device=$(xcrun simctl list devices "${device_name}" available | grep "${device_name} (")
re='\(([-0-9A-Fa-f]+)\)'
[[ $device =~ $re ]] || exit 1
xcodebuild -workspace example.xcworkspace -scheme example -destination "platform=iOS Simulator,id=${BASH_REMATCH[1]}" CODE_SIGNING_ALLOWED=NO COMPILER_INDEX_STORE_ENABLE=NO build
set -o pipefail && xcodebuild -workspace example.xcworkspace -scheme example -destination "platform=iOS Simulator,id=${BASH_REMATCH[1]}" CODE_SIGNING_ALLOWED=NO COMPILER_INDEX_STORE_ENABLE=NO build | xcbeautify --renderer github-actions
working-directory: example/ios


Expand Down
147 changes: 125 additions & 22 deletions example/ios/Podfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -3,13 +3,6 @@ PODS:
- CocoaAsyncSocket (7.6.5)
- DoubleConversion (1.1.6)
- FBLazyVector (0.73.5)
- FBReactNativeSpec (0.73.5):
- RCT-Folly (= 2022.05.16.00)
- RCTRequired (= 0.73.5)
- RCTTypeSafety (= 0.73.5)
- React-Core (= 0.73.5)
- React-jsi (= 0.73.5)
- ReactCommon/turbomodule/core (= 0.73.5)
- Flipper (0.201.0):
- Flipper-Folly (~> 2.6)
- Flipper-Boost-iOSX (1.76.0.1.11)
Expand Down Expand Up @@ -116,17 +109,21 @@ PODS:
- React-callinvoker (0.73.5)
- React-Codegen (0.73.5):
- DoubleConversion
- FBReactNativeSpec
- glog
- hermes-engine
- RCT-Folly
- RCTRequired
- RCTTypeSafety
- React-Core
- React-debug
- React-Fabric
- React-FabricImage
- React-graphics
- React-jsi
- React-jsiexecutor
- React-NativeModulesApple
- React-rncore
- React-rendererdebug
- React-utils
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- React-Core (0.73.5):
Expand Down Expand Up @@ -939,19 +936,71 @@ PODS:
- React-jsi (= 0.73.5)
- React-perflogger (= 0.73.5)
- React-jsinspector (0.73.5)
- React-jsitracing (0.73.5):
- React-jsi
- React-logger (0.73.5):
- glog
- React-Mapbuffer (0.73.5):
- glog
- React-debug
- react-native-pager-view (6.2.3):
- react-native-pager-view (6.3.0):
- glog
- hermes-engine
- RCT-Folly (= 2022.05.16.00)
- RCTRequired
- RCTTypeSafety
- React-Codegen
- React-Core
- React-debug
- React-Fabric
- React-graphics
- React-ImageManager
- React-NativeModulesApple
- React-RCTFabric
- React-rendererdebug
- React-utils
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- Yoga
- react-native-slider (4.5.0):
- glog
- hermes-engine
- RCT-Folly (= 2022.05.16.00)
- RCTRequired
- RCTTypeSafety
- React-Codegen
- React-Core
- React-debug
- React-Fabric
- React-graphics
- React-ImageManager
- react-native-slider/common (= 4.5.0)
- React-NativeModulesApple
- React-RCTFabric
- React-rendererdebug
- React-utils
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- Yoga
- react-native-slider/common (4.5.0):
- glog
- hermes-engine
- RCT-Folly (= 2022.05.16.00)
- RCTRequired
- RCTTypeSafety
- React-Codegen
- React-Core
- React-debug
- React-Fabric
- React-graphics
- React-ImageManager
- React-NativeModulesApple
- React-RCTFabric
- React-rendererdebug
- React-utils
- ReactCommon/turbomodule/bridging
- ReactCommon/turbomodule/core
- Yoga
- React-nativeconfig (0.73.5)
- React-NativeModulesApple (0.73.5):
- glog
Expand Down Expand Up @@ -980,13 +1029,21 @@ PODS:
- RCTTypeSafety
- React-Core
- React-CoreModules
- React-debug
- React-Fabric
- React-graphics
- React-hermes
- React-nativeconfig
- React-NativeModulesApple
- React-RCTFabric
- React-RCTImage
- React-RCTNetwork
- React-rendererdebug
- React-RuntimeApple
- React-RuntimeCore
- React-RuntimeHermes
- React-runtimescheduler
- React-utils
- ReactCommon
- React-RCTBlob (0.73.5):
- hermes-engine
Expand Down Expand Up @@ -1064,8 +1121,42 @@ PODS:
- RCT-Folly (= 2022.05.16.00)
- React-debug
- React-rncore (0.73.5)
- React-RuntimeApple (0.73.5):
- hermes-engine
- RCT-Folly/Fabric (= 2022.05.16.00)
- React-callinvoker
- React-Core/Default
- React-CoreModules
- React-cxxreact
- React-jserrorhandler
- React-jsi
- React-jsiexecutor
- React-Mapbuffer
- React-NativeModulesApple
- React-RCTFabric
- React-RuntimeCore
- React-runtimeexecutor
- React-RuntimeHermes
- React-utils
- React-RuntimeCore (0.73.5):
- glog
- hermes-engine
- RCT-Folly/Fabric (= 2022.05.16.00)
- React-cxxreact
- React-jserrorhandler
- React-jsi
- React-jsiexecutor
- React-runtimeexecutor
- React-runtimescheduler
- React-runtimeexecutor (0.73.5):
- React-jsi (= 0.73.5)
- React-RuntimeHermes (0.73.5):
- hermes-engine
- RCT-Folly/Fabric (= 2022.05.16.00)
- React-jsi
- React-jsitracing
- React-nativeconfig
- React-utils
- React-runtimescheduler (0.73.5):
- glog
- hermes-engine
Expand Down Expand Up @@ -1126,7 +1217,6 @@ DEPENDENCIES:
- boost (from `../node_modules/react-native/third-party-podspecs/boost.podspec`)
- DoubleConversion (from `../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec`)
- FBLazyVector (from `../node_modules/react-native/Libraries/FBLazyVector`)
- FBReactNativeSpec (from `../node_modules/react-native/React/FBReactNativeSpec`)
- Flipper (= 0.201.0)
- Flipper-Boost-iOSX (= 1.76.0.1.11)
- Flipper-DoubleConversion (= 3.2.0.1)
Expand Down Expand Up @@ -1173,6 +1263,7 @@ DEPENDENCIES:
- React-jsi (from `../node_modules/react-native/ReactCommon/jsi`)
- React-jsiexecutor (from `../node_modules/react-native/ReactCommon/jsiexecutor`)
- React-jsinspector (from `../node_modules/react-native/ReactCommon/jsinspector-modern`)
- React-jsitracing (from `../node_modules/react-native/ReactCommon/hermes/executor/`)
- React-logger (from `../node_modules/react-native/ReactCommon/logger`)
- React-Mapbuffer (from `../node_modules/react-native/ReactCommon`)
- react-native-pager-view (from `../node_modules/react-native-pager-view`)
Expand All @@ -1193,7 +1284,10 @@ DEPENDENCIES:
- React-RCTVibration (from `../node_modules/react-native/Libraries/Vibration`)
- React-rendererdebug (from `../node_modules/react-native/ReactCommon/react/renderer/debug`)
- React-rncore (from `../node_modules/react-native/ReactCommon`)
- React-RuntimeApple (from `../node_modules/react-native/ReactCommon/react/runtime/platform/ios`)
- React-RuntimeCore (from `../node_modules/react-native/ReactCommon/react/runtime`)
- React-runtimeexecutor (from `../node_modules/react-native/ReactCommon/runtimeexecutor`)
- React-RuntimeHermes (from `../node_modules/react-native/ReactCommon/react/runtime`)
- React-runtimescheduler (from `../node_modules/react-native/ReactCommon/react/renderer/runtimescheduler`)
- React-utils (from `../node_modules/react-native/ReactCommon/react/utils`)
- ReactCommon/turbomodule/core (from `../node_modules/react-native/ReactCommon`)
Expand Down Expand Up @@ -1222,8 +1316,6 @@ EXTERNAL SOURCES:
:podspec: "../node_modules/react-native/third-party-podspecs/DoubleConversion.podspec"
FBLazyVector:
:path: "../node_modules/react-native/Libraries/FBLazyVector"
FBReactNativeSpec:
:path: "../node_modules/react-native/React/FBReactNativeSpec"
glog:
:podspec: "../node_modules/react-native/third-party-podspecs/glog.podspec"
hermes-engine:
Expand Down Expand Up @@ -1267,6 +1359,8 @@ EXTERNAL SOURCES:
:path: "../node_modules/react-native/ReactCommon/jsiexecutor"
React-jsinspector:
:path: "../node_modules/react-native/ReactCommon/jsinspector-modern"
React-jsitracing:
:path: "../node_modules/react-native/ReactCommon/hermes/executor/"
React-logger:
:path: "../node_modules/react-native/ReactCommon/logger"
React-Mapbuffer:
Expand Down Expand Up @@ -1307,8 +1401,14 @@ EXTERNAL SOURCES:
:path: "../node_modules/react-native/ReactCommon/react/renderer/debug"
React-rncore:
:path: "../node_modules/react-native/ReactCommon"
React-RuntimeApple:
:path: "../node_modules/react-native/ReactCommon/react/runtime/platform/ios"
React-RuntimeCore:
:path: "../node_modules/react-native/ReactCommon/react/runtime"
React-runtimeexecutor:
:path: "../node_modules/react-native/ReactCommon/runtimeexecutor"
React-RuntimeHermes:
:path: "../node_modules/react-native/ReactCommon/react/runtime"
React-runtimescheduler:
:path: "../node_modules/react-native/ReactCommon/react/renderer/runtimescheduler"
React-utils:
Expand All @@ -1323,7 +1423,6 @@ SPEC CHECKSUMS:
CocoaAsyncSocket: 065fd1e645c7abab64f7a6a2007a48038fdc6a99
DoubleConversion: fea03f2699887d960129cc54bba7e52542b6f953
FBLazyVector: 56e0e498dbb513b96c40bac6284729ba4e62672d
FBReactNativeSpec: 146c741a3f40361f6bc13a4ba284678cbedb5881
Flipper: c7a0093234c4bdd456e363f2f19b2e4b27652d44
Flipper-Boost-iOSX: fd1e2b8cbef7e662a122412d7ac5f5bea715403c
Flipper-DoubleConversion: 2dc99b02f658daf147069aad9dbd29d8feb06d30
Expand All @@ -1342,7 +1441,7 @@ SPEC CHECKSUMS:
RCTTypeSafety: 50efabe2b115c11ed03fbf3fd79e2f163ddb5d7c
React: 84221d5e0ce297bc57c4b6af539a62d812d89f10
React-callinvoker: 5d17577ecc7f784535ebedf3aad4bcbf8f4b5117
React-Codegen: 857e7984fc277aadde2a7a427288b6918ece7b2b
React-Codegen: 622eb4971c887cd198ce55346860635430fe8f72
React-Core: 8e782e7e24c7843871a0d9c3c8d7c5b3ebb73832
React-CoreModules: 7875ee247e3e6e0e683b52cd1cdda1b71618bd55
React-cxxreact: 788cd771c6e94d44f8d472fdfae89b67226067ea
Expand All @@ -1356,33 +1455,37 @@ SPEC CHECKSUMS:
React-jsi: 1d59d0a148c76641ac577729e0268bafa494152c
React-jsiexecutor: 262b66928ad948491d03fd328bb5b822cce94647
React-jsinspector: 32db5e364bcae8fca8cdf8891830636275add0c5
React-jsitracing: 42912570ecc01b07e29894a1a05a54f270e683ce
React-logger: 0331362115f0f5b392bd7ed14636d1a3ea612479
React-Mapbuffer: 7c35cd53a22d0be04d3f26f7881c7fb7dd230216
react-native-pager-view: d5f3adb58a4e6e0d200055e9a4afdcda9b9022ce
react-native-slider: 09e5a8b7e766d3b5ae24ec15c5c4ec2679ca0f8c
react-native-pager-view: b5c1234655bc56d201f8ea1ce887f6cabeafd61b
react-native-slider: e4be63dee7fb65a2318468de34aa9a423a2112f6
React-nativeconfig: 1166714a4f7ea57a0df5c2cb44fbc70f98d580f9
React-NativeModulesApple: 726664e9829eb5eed8170241000e46ead269a05f
React-perflogger: 0dd9f1725d55f8264b81efadd373fe1d9cca7dc2
React-RCTActionSheet: 05656d2102b0d0a2676d58bad4d80106af5367b2
React-RCTAnimation: 6c66beae98730fb7615df28caf651e295f2401e5
React-RCTAppDelegate: 891b80c596fffcb3f90431739495d606a9a0d610
React-RCTAppDelegate: d78fa6bcb3c823201b77bb86a967a0efd2dd4eed
React-RCTBlob: 8ecee445ec5fa9ed8a8621a136183c1045165100
React-RCTFabric: f291e06bc63fef26cdd105537bae5c6a8d3bdca8
React-RCTFabric: 43929bf7439754d75bd9a88f97c990bfb98c90fd
React-RCTImage: 585b16465146cb839da02f3179ce7cb19d332642
React-RCTLinking: 09ba11f7df62946e7ddca1b51aa3bf47b230e008
React-RCTNetwork: e070f8d2fca60f1e9571936ce54d165e77129e76
React-RCTSettings: b08c7ff191f0a5421aab198ea1086c9a8d513bde
React-RCTText: f6cc5a3cf0f1a4f3d1256657dca1025e4cfe45e0
React-RCTVibration: d9948962139f9924ef87f23ab240e045e496213b
React-rendererdebug: ee05480666415f7a76e6cf0a7a50363423f44809
React-rncore: 010565651e9cf2e4fac9517a348446789dd55e01
React-rncore: a93b592afe8ff28029a3c4009e52da14a0516c90
React-RuntimeApple: 95172dcfb260834a2bf1c8302b27ef0bc276e079
React-RuntimeCore: 404f636cf4144ead99c1bf46fad1cbecede27876
React-runtimeexecutor: 56f562a608056fb0c1711d900a992e26f375d817
React-RuntimeHermes: 4a505ba2d60c4c8523d28f1d69dd5d115d81a6d3
React-runtimescheduler: 814b644a5f456c7df1fba7bcd9914707152527c6
React-utils: 987a4526a2fc0acdfaf87888adfe0bf9d0452066
ReactCommon: 2947b0bffd82ea0e58ca7928881152d4c6dae9af
SocketRocket: f32cd54efbe0f095c4d7594881e52619cfe80b17
Yoga: a716eea57d0d3430219c0a5a233e1e93ee931eb7
Yoga: 9e6a04eacbd94f97d94577017e9f23b3ab41cf6c

PODFILE CHECKSUM: c25872b6c8e4ec17e5338750fa46de6bf56fcdce

COCOAPODS: 1.14.2
COCOAPODS: 1.14.3
9 changes: 1 addition & 8 deletions package/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ android {
sourceSets {
main {
if (isNewArchitectureEnabled()) {
java.srcDirs += ['src/newarch']
java.srcDirs += ['src/newarch', "${project.buildDir}/generated/source/codegen/java"]
} else {
java.srcDirs += ['src/oldarch']
}
Expand All @@ -73,10 +73,3 @@ dependencies {
api 'com.facebook.react:react-native:+'
}

if (isNewArchitectureEnabled()) {
react {
jsRootDir = file("../src")
libraryName = "ReactSlider"
codegenJavaPackageName = "com.reactnativecommunity.slider"
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -134,8 +134,16 @@ public static void setAccessibilityIncrements(ReactSlider view, ReadableArray ac
view.setAccessibilityIncrements(stringList);
}

public static Map getExportedCustomDirectEventTypeConstants() {
return MapBuilder.of(ReactSlidingCompleteEvent.EVENT_NAME, MapBuilder.of("registrationName", "onRNCSliderSlidingComplete"),
ReactSlidingStartEvent.EVENT_NAME, MapBuilder.of("registrationName", "onRNCSliderSlidingStart"));
public static Map<String, Object> getExportedCustomBubblingEventTypeConstants() {
return MapBuilder.of(
ReactSliderEvent.EVENT_NAME, MapBuilder.of("registrationName", ReactSliderEvent.EVENT_NAME)
);
}

public static Map<String, Object> getExportedCustomDirectEventTypeConstants() {
return MapBuilder.of(
ReactSlidingStartEvent.EVENT_NAME, MapBuilder.of("registrationName", ReactSlidingStartEvent.EVENT_NAME),
ReactSlidingCompleteEvent.EVENT_NAME, MapBuilder.of("registrationName", ReactSlidingCompleteEvent.EVENT_NAME)
);
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
*/
public class ReactSlidingCompleteEvent extends Event<ReactSlidingCompleteEvent> {

public static final String EVENT_NAME = "topSlidingComplete";
public static final String EVENT_NAME = "onRNCSliderSlidingComplete";

private final double mValue;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@
*/

public class ReactSlidingStartEvent extends Event<ReactSlidingStartEvent> {
public static final String EVENT_NAME = "topSlidingStart";
public static final String EVENT_NAME = "onRNCSliderSlidingStart";

private final double mValue;

Expand Down Expand Up @@ -58,4 +58,4 @@ private WritableMap serializeEventData() {
return eventData;
}

}
}
Loading
Loading