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

feat: React Native 0.71.14 with Xcode 15 stable #7114

Merged
merged 59 commits into from Oct 30, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
59 commits
Select commit Hold shift + click to select a range
0d199d7
Bump Branch IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12
leotm Sep 4, 2023
acb6d40
Bump MixPanel IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12
leotm Sep 4, 2023
07ed7d4
Upgrade RN from 0.71.6 to 0.71.13 (npm)
leotm Sep 4, 2023
dbd5f04
Disable Flipper in Podfile and update lockfile (RN 0.71.13)
leotm Sep 4, 2023
1e432e0
Add NO_FLIPPER=1 to pod:install script
leotm Sep 4, 2023
14d18a6
Account for NO_FLIPPER (react-native-flipper) in react-native.config.js
leotm Sep 4, 2023
a7ec533
Update Sentry (npm)
leotm Sep 4, 2023
0cbd66f
Revert "Update Sentry (npm)"
leotm Sep 29, 2023
644feae
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
leotm Sep 29, 2023
6e0da25
Bump RN patch from 0.71.6 to 0.71.13
leotm Sep 29, 2023
120eae9
Revert "Bump MixPanel IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12"
leotm Oct 2, 2023
7acab31
Revert "Bump Branch IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12"
leotm Oct 2, 2023
b054c2c
pod repo update && pod install
leotm Oct 2, 2023
a09d517
Fix Xcode 15 error: Unable to load ...frameworks-Debug-input-files.xc…
leotm Oct 2, 2023
1c9fc2b
Bump Branch IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12
leotm Oct 2, 2023
71a8b0e
Bump MixPanel IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12
leotm Oct 2, 2023
8daa624
Bump BitRise stack from osx-xcode-14.2.x-ventura to osx-xcode-15.0.x
leotm Oct 2, 2023
66d3ce6
Revert "Bump MixPanel IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12"
leotm Oct 6, 2023
de90dbd
Revert "Bump Branch IPHONEOS_DEPLOYMENT_TARGETs from 8 to 12"
leotm Oct 6, 2023
ffaa984
Fix Podfile react_native_post_install whitespace indentation
leotm Oct 6, 2023
2ca8f1d
Disable Podfile custom react_native_post_install
leotm Oct 6, 2023
cf1c1e3
Fix FlipperKit files not found
leotm Oct 6, 2023
d0235cd
Fix FlipperKit use of undeclared identifiers
leotm Oct 6, 2023
a2c80a3
Update mixpanel-iphone git submodule commit hash
leotm Oct 11, 2023
26a6c52
update .gitmodule file with mixpanel-iphone fork. remove duplicate su…
jpcloureiro Oct 11, 2023
527dfd1
Update mixpanel-iphone git submodule commit hash
leotm Oct 11, 2023
b4b9908
update .gitmodule file with branch-ios-sdk fork. remove duplicate sub…
leotm Oct 11, 2023
5b399e8
Update podfile.lock
leotm Oct 13, 2023
f91973b
Remove duplicate Podfile flipper_config comment
leotm Oct 13, 2023
f773d23
Remove stale Podfile flipper_post_install comment
leotm Oct 13, 2023
d1213f1
Refactor Podfile react_native_post_install
leotm Oct 13, 2023
823c262
Reenable Podfile flipper_config
leotm Oct 14, 2023
77f5f9b
Move Podfile flipper_config comment to correct location
leotm Oct 14, 2023
aaba386
Remove FB_SONARKIT_ENABLED flag from xcodeproj release build settings
leotm Oct 14, 2023
83be066
Refactor xcodeproj FB_SONARKIT_ENABLED flag to macro
leotm Oct 14, 2023
a2d850f
Remove FB_SONARKIT_ENABLED flag from xcodeproj debug build settings
leotm Oct 14, 2023
8a41868
Reenable AppDelegate.m Flipper imports
leotm Oct 14, 2023
f6dec93
Remove NO_FLIPPER=1 from package.json setup script
leotm Oct 14, 2023
1c24f56
Add back Flipper to Podfile and xcodeproj
leotm Oct 14, 2023
544cd1c
Revert "Fix Xcode 15 error: Unable to load ...frameworks-Debug-input-…
leotm Oct 15, 2023
4bb7c33
Add Podfile react_native_post_install comment
leotm Oct 16, 2023
40934e6
Upgrade React Native from 0.71.13 to 0.71.14
leotm Oct 17, 2023
67e6be0
Bump Detox from 20.11.0 to 20.11.4
leotm Oct 17, 2023
aff0b90
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
leotm Oct 17, 2023
8055db8
Remove unneeded extra spaces in build scripts
leotm Oct 18, 2023
f079514
Bump Detox iOS sim from iPhone 12 Pro to iPhone 13 Pro for Xcode 15
leotm Oct 18, 2023
ad16fd9
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
leotm Oct 18, 2023
5006b1a
Update build scripts from iPhone 12 Pro to iPhone 13 Pro
leotm Oct 19, 2023
c066fc0
Update release build scripts from iPhone 12 Pro to iPhone 13 Pro
leotm Oct 19, 2023
b5a3f54
Update ReadMe iPhone 12 Pro refs to iPhone 13 Pro
leotm Oct 19, 2023
ed91ac6
Update ReadMe Xcode 14.2 to Xcode 15.0
leotm Oct 19, 2023
fbde137
Update ReadMe Xcode 15.0 to Xcode 15.0.1
leotm Oct 19, 2023
8fbec65
Bump metro devDep from 0.72.3 to 0.73.10
leotm Oct 25, 2023
44da518
Remove stale Device util static method isIphone12
leotm Oct 25, 2023
8e5c7d5
Bump Browserstack iPhone 12 Pro devices to iPhone 13 Pro
leotm Oct 25, 2023
8b38ade
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
leotm Oct 25, 2023
c6bbee3
Fix lint error remove unused var
leotm Oct 25, 2023
7e08166
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
legobeat Oct 25, 2023
59de56a
Merge branch 'main' into react-native-0.71.13-xcode-15-beta-8
leotm Oct 29, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion .detoxrc.js
Expand Up @@ -56,7 +56,7 @@ module.exports = {
'ios.simulator': {
type: 'ios.simulator',
device: {
type: 'iPhone 12 Pro',
type: 'iPhone 13 Pro',
leotm marked this conversation as resolved.
Show resolved Hide resolved
},
},
'android.bitrise.emulator': {
Expand Down
8 changes: 2 additions & 6 deletions .gitmodules
@@ -1,10 +1,6 @@
[submodule "ios/mixpanel-iphone"]
path = ios/mixpanel-iphone
url = https://github.com/mixpanel/mixpanel-iphone
url = https://github.com/metamask/mixpanel-iphone
[submodule "ios/branch-ios-sdk"]
path = ios/branch-ios-sdk
url = https://github.com/BranchMetrics/ios-branch-deep-linking-attribution

[submodule "ios/mixpanel-iphone"]
path = ios/mixpanel-iphone
url = https://github.com/mixpanel/mixpanel-iphone
leotm marked this conversation as resolved.
Show resolved Hide resolved
url = https://github.com/metamask/ios-branch-deep-linking-attribution
leotm marked this conversation as resolved.
Show resolved Hide resolved
1 change: 1 addition & 0 deletions Gemfile
Expand Up @@ -4,3 +4,4 @@ source 'https://rubygems.org'
ruby '>= 3.0.0'

gem 'cocoapods', '>= 1.11.3'
gem 'activesupport', '>= 6.1.7.3', '< 7.1.0'
6 changes: 3 additions & 3 deletions README.md
Expand Up @@ -37,7 +37,7 @@ The code is built using React-Native and running code locally requires a Mac or
```

- Install the shared [React Native dependencies](https://reactnative.dev/docs/environment-setup#installing-dependencies) (`React Native CLI`, _not_ `Expo CLI`)
- XCode version `14.2` or below
- XCode version `15.0.1` or below

- Install [cocoapods](https://guides.cocoapods.org/using/getting-started.html) by running:

Expand Down Expand Up @@ -90,7 +90,7 @@ The code is built using React-Native and running code locally requires a Mac or
- [React Native Getting Started - iOS](https://reactnative.dev/docs/environment-setup#installing-dependencies) _(React Native CLI Quickstart -> [your OS] -> iOS)_
- Install the correct simulator
- **iOS OS Version:** Latest, unless told otherwise
- **Device:** iPhone 12 Pro
- **Device:** iPhone 13 Pro



Expand Down Expand Up @@ -237,7 +237,7 @@ Prerequisites for running tests:
- Make sure to install `detox-cli` by referring to the instructions mentioned [here](https://wix.github.io/Detox/docs/introduction/getting-started/#detox-prerequisites).
- Additionally, install `applesimutils` by following the guidelines provided [here](https://github.com/wix/AppleSimulatorUtils).
- Before running any tests, it's recommended to refer to the `iOS section` above and check the latest simulator device specified under `Install the correct simulator`.
- The default device for iOS is the iPhone 12 Pro and Android the Pixel 5. Ensure you have these set up.
- The default device for iOS is the iPhone 13 Pro and Android the Pixel 5. Ensure you have these set up.
- Make sure that Metro is running. Use this command to launch the metro server:

```bash
Expand Down
8 changes: 1 addition & 7 deletions app/util/device/index.js
@@ -1,7 +1,7 @@
'use strict';

import { Dimensions, Platform } from 'react-native';
import { getModel, hasNotch, getApiLevel } from 'react-native-device-info';
import { hasNotch, getApiLevel } from 'react-native-device-info';

export default class Device {
static getDeviceWidth() {
Expand Down Expand Up @@ -86,12 +86,6 @@ export default class Device {
return this.getDeviceHeight() > 736;
}

static isIphone12() {
const model = getModel();
const models = ['iPhone 12', 'iPhone 12 Pro', 'iPhone 12 Pro Max'];
return models.includes(model);
}

static hasNotch() {
return hasNotch();
}
Expand Down
2 changes: 1 addition & 1 deletion bitrise.yml
Expand Up @@ -815,7 +815,7 @@ app:
IOS_APP_LINK: ''
meta:
bitrise.io:
stack: osx-xcode-14.2.x-ventura
stack: osx-xcode-15.0.x
leotm marked this conversation as resolved.
Show resolved Hide resolved
machine_type_id: g2-m1-max.5core
trigger_map:
- push_branch: release/*
Expand Down
79 changes: 47 additions & 32 deletions ios/MetaMask.xcodeproj/project.pbxproj
Expand Up @@ -557,8 +557,8 @@
13B07F8E1A680F5B00A75B9A /* Resources */,
00DD1BFF1BD5951E006B06BC /* Bundle React Native code and images */,
15ACCA0022655C3A0063978B /* Embed Frameworks */,
3263892F1BBB809723CB4024 /* [CP] Embed Pods Frameworks */,
802588CED3FC487A5D5263F0 /* [CP] Copy Pods Resources */,
C77EFC60F19570875F8DB3BD /* [CP] Embed Pods Frameworks */,
);
buildRules = (
);
Expand All @@ -582,8 +582,8 @@
B339FF0F289ABD70001B89FB /* Resources */,
B339FF2F289ABD70001B89FB /* Bundle React Native code and images */,
B339FF30289ABD70001B89FB /* Embed Frameworks */,
C9BADD20B98A4D13195A3200 /* [CP] Embed Pods Frameworks */,
8EF10BB14629809332947E5D /* [CP] Copy Pods Resources */,
EB3465C9579A347237ADD532 /* [CP] Embed Pods Frameworks */,
);
buildRules = (
);
Expand Down Expand Up @@ -815,23 +815,6 @@
shellPath = /bin/sh;
shellScript = "if [ -e ../.ios.env ]\nthen\n cp -rf ../.ios.env debug.xcconfig\n cp -rf ../.ios.env release.xcconfig\nelse\n cp -rf ../.ios.env.example debug.xcconfig\n cp -rf ../.ios.env.example release.xcconfig\nfi\n\n";
};
3263892F1BBB809723CB4024 /* [CP] Embed Pods Frameworks */ = {
isa = PBXShellScriptBuildPhase;
buildActionMask = 2147483647;
files = (
);
inputFileListPaths = (
"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks-${CONFIGURATION}-input-files.xcfilelist",
);
name = "[CP] Embed Pods Frameworks";
outputFileListPaths = (
"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks-${CONFIGURATION}-output-files.xcfilelist",
);
runOnlyForDeploymentPostprocessing = 0;
shellPath = /bin/sh;
shellScript = "\"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks.sh\"\n";
showEnvVarsInLog = 0;
};
leotm marked this conversation as resolved.
Show resolved Hide resolved
54113A1E76112B187F92C803 /* [CP] Check Pods Manifest.lock */ = {
isa = PBXShellScriptBuildPhase;
buildActionMask = 2147483647;
Expand Down Expand Up @@ -945,7 +928,24 @@
shellPath = /bin/sh;
shellScript = "if [[ -s \"$HOME/.nvm/nvm.sh\" ]]; then\n. \"$HOME/.nvm/nvm.sh\"\nelif [[ -x \"$(command -v brew)\" && -s \"$(brew --prefix nvm)/nvm.sh\" ]]; then\n. \"$(brew --prefix nvm)/nvm.sh\"\nfi\n\nexport NODE_BINARY=$(which node)\n\nif [[ -s \"$HOME/.nvm/nvm.sh\" ]]; then\n. \"$HOME/.nvm/nvm.sh\"\nelif [[ -x \"$(command -v brew)\" && -s \"$(brew --prefix nvm)/nvm.sh\" ]]; then\n. \"$(brew --prefix nvm)/nvm.sh\"\nfi\n\nif [ ! -e \"${SENTRY_PROPERTIES}\" ]; then\n export SENTRY_PROPERTIES=../sentry.properties\nfi\n\n\nif [ \"$PLATFORM_NAME\" == \"iphonesimulator\" ] && [ \"$CONFIGURATION\" == \"Release\" ]; then\n ../node_modules/react-native/scripts/react-native-xcode.sh\nelse\n ../node_modules/@sentry/cli/bin/sentry-cli react-native xcode \\\n ../node_modules/react-native/scripts/react-native-xcode.sh\nfi\n";
};
C9BADD20B98A4D13195A3200 /* [CP] Embed Pods Frameworks */ = {
C77EFC60F19570875F8DB3BD /* [CP] Embed Pods Frameworks */ = {
isa = PBXShellScriptBuildPhase;
buildActionMask = 2147483647;
files = (
);
inputFileListPaths = (
"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks-${CONFIGURATION}-input-files.xcfilelist",
);
name = "[CP] Embed Pods Frameworks";
outputFileListPaths = (
"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks-${CONFIGURATION}-output-files.xcfilelist",
);
runOnlyForDeploymentPostprocessing = 0;
shellPath = /bin/sh;
shellScript = "\"${PODS_ROOT}/Target Support Files/Pods-MetaMask/Pods-MetaMask-frameworks.sh\"\n";
showEnvVarsInLog = 0;
};
EB3465C9579A347237ADD532 /* [CP] Embed Pods Frameworks */ = {
isa = PBXShellScriptBuildPhase;
buildActionMask = 2147483647;
files = (
Expand Down Expand Up @@ -1071,14 +1071,11 @@
"$(inherited)",
"@executable_path/Frameworks",
);
LIBRARY_SEARCH_PATHS = "$(inherited)";
LIBRARY_SEARCH_PATHS = "$(SDKROOT)/usr/lib/swift$(inherited)";
LLVM_LTO = YES;
MARKETING_VERSION = 7.9.0;
ONLY_ACTIVE_ARCH = YES;
OTHER_CFLAGS = (
"$(inherited)",
"-DFB_SONARKIT_ENABLED=1",
leotm marked this conversation as resolved.
Show resolved Hide resolved
);
OTHER_CFLAGS = "$(inherited)";
OTHER_LDFLAGS = (
"$(inherited)",
"-ObjC",
Expand Down Expand Up @@ -1137,14 +1134,11 @@
"$(inherited)",
"@executable_path/Frameworks",
);
LIBRARY_SEARCH_PATHS = "$(inherited)";
LIBRARY_SEARCH_PATHS = "$(SDKROOT)/usr/lib/swift$(inherited)";
LLVM_LTO = YES;
MARKETING_VERSION = 7.9.0;
ONLY_ACTIVE_ARCH = NO;
OTHER_CFLAGS = (
"$(inherited)",
"-DFB_SONARKIT_ENABLED=1",
leotm marked this conversation as resolved.
Show resolved Hide resolved
);
OTHER_CFLAGS = "$(inherited)";
OTHER_LDFLAGS = (
"$(inherited)",
"-ObjC",
Expand All @@ -1165,7 +1159,7 @@
baseConfigurationReference = 15FDD82721B7642B006B7C35 /* debug.xcconfig */;
buildSettings = {
ALWAYS_SEARCH_USER_PATHS = NO;
CLANG_CXX_LANGUAGE_STANDARD = "gnu++0x";
CLANG_CXX_LANGUAGE_STANDARD = "c++17";
CLANG_CXX_LIBRARY = "libc++";
CLANG_ENABLE_MODULES = YES;
CLANG_ENABLE_OBJC_ARC = YES;
Expand All @@ -1183,12 +1177,14 @@
ENABLE_BITCODE = NO;
ENABLE_STRICT_OBJC_MSGSEND = YES;
ENABLE_TESTABILITY = YES;
"EXCLUDED_ARCHS[sdk=iphonesimulator*]" = "";
GCC_C_LANGUAGE_STANDARD = gnu99;
GCC_DYNAMIC_NO_PIC = NO;
GCC_OPTIMIZATION_LEVEL = 0;
GCC_PREPROCESSOR_DEFINITIONS = (
"DEBUG=1",
"$(inherited)",
_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION,
);
GCC_SYMBOLS_PRIVATE_EXTERN = NO;
GCC_WARN_64_TO_32_BIT_CONVERSION = YES;
Expand All @@ -1200,6 +1196,12 @@
IPHONEOS_DEPLOYMENT_TARGET = 11.0;
MTL_ENABLE_DEBUG_INFO = YES;
ONLY_ACTIVE_ARCH = YES;
OTHER_LDFLAGS = (
"$(inherited)",
"-Wl",
"-ld_classic",
);
REACT_NATIVE_PATH = "${PODS_ROOT}/../../node_modules/react-native";
SDKROOT = iphoneos;
};
name = Debug;
Expand All @@ -1208,7 +1210,7 @@
isa = XCBuildConfiguration;
buildSettings = {
ALWAYS_SEARCH_USER_PATHS = NO;
CLANG_CXX_LANGUAGE_STANDARD = "gnu++0x";
CLANG_CXX_LANGUAGE_STANDARD = "c++17";
CLANG_CXX_LIBRARY = "libc++";
CLANG_ENABLE_MODULES = YES;
CLANG_ENABLE_OBJC_ARC = YES;
Expand All @@ -1226,7 +1228,12 @@
ENABLE_BITCODE = NO;
ENABLE_NS_ASSERTIONS = NO;
ENABLE_STRICT_OBJC_MSGSEND = YES;
"EXCLUDED_ARCHS[sdk=iphonesimulator*]" = "";
GCC_C_LANGUAGE_STANDARD = gnu99;
GCC_PREPROCESSOR_DEFINITIONS = (
"$(inherited)",
_LIBCPP_ENABLE_CXX17_REMOVED_UNARY_BINARY_FUNCTION,
);
GCC_WARN_64_TO_32_BIT_CONVERSION = YES;
GCC_WARN_ABOUT_RETURN_TYPE = YES_ERROR;
GCC_WARN_UNDECLARED_SELECTOR = YES;
Expand All @@ -1235,6 +1242,12 @@
GCC_WARN_UNUSED_VARIABLE = YES;
IPHONEOS_DEPLOYMENT_TARGET = 11.0;
MTL_ENABLE_DEBUG_INFO = NO;
OTHER_LDFLAGS = (
"$(inherited)",
"-Wl",
"-ld_classic",
);
REACT_NATIVE_PATH = "${PODS_ROOT}/../../node_modules/react-native";
SDKROOT = iphoneos;
VALIDATE_PRODUCT = YES;
};
Expand Down Expand Up @@ -1284,6 +1297,7 @@
"@executable_path/Frameworks",
);
LIBRARY_SEARCH_PATHS = (
"$(SDKROOT)/usr/lib/swift",
"$(inherited)",
"\"$(SRCROOT)/MetaMask/System/Library/Frameworks\"",
);
Expand Down Expand Up @@ -1353,6 +1367,7 @@
"@executable_path/Frameworks",
);
LIBRARY_SEARCH_PATHS = (
"$(SDKROOT)/usr/lib/swift",
"$(inherited)",
"\"$(SRCROOT)/MetaMask/System/Library/Frameworks\"",
);
Expand Down