Skip to content

Commit

Permalink
Address feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
siddarthkay committed May 24, 2023
1 parent de4b56c commit 8e99442
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 43 deletions.
7 changes: 0 additions & 7 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -231,13 +231,6 @@ jsbundle: export BUILD_ENV ?= prod
jsbundle: ##@build Build JavaScript and Clojurescript bundle for iOS and Android
nix/scripts/build.sh targets.mobile.jsbundle

release-ios-pr: export TARGET := ios
release-ios-pr: export BUILD_ENV ?= prod
release-ios-pr: watchman-clean ##@build Build release PR for iOS release
@git clean -dxf -f target/ios && \
$(MAKE) jsbundle-ios && \
xcodebuild -workspace ios/StatusIm.xcworkspace -scheme StatusImPR -configuration Release -destination 'generic/platform=iOS' -UseModernBuildSystem=N clean archive | xcpretty --no-color

#--------------
# status-go lib
#--------------
Expand Down
22 changes: 0 additions & 22 deletions android/app/src/main/java/im/status/ethereum/MainActivity.java
Original file line number Diff line number Diff line change
Expand Up @@ -257,26 +257,4 @@ public void onRequestPermissionsResult(int requestCode, String[] permissions, in
mPermissionListener = null;
}
}
/**
* Returns the instance of the {@link ReactActivityDelegate}. There the RootView is created and
* you can specify the rendered you wish to use (Fabric or the older renderer).
*/
// @Override
// protected ReactActivityDelegate createReactActivityDelegate() {
// return new MainActivityDelegate(this, getMainComponentName());
// }
//
// public static class MainActivityDelegate extends ReactActivityDelegate {
// public MainActivityDelegate(ReactActivity activity, String mainComponentName) {
// super(activity, mainComponentName);
// }
//
// @Override
// protected ReactRootView createRootView() {
// ReactRootView reactRootView = new ReactRootView(getContext());
// // If you opted-in for the New Architecture, we enable the Fabric Renderer.
// reactRootView.setIsFabric(BuildConfig.IS_NEW_ARCHITECTURE_ENABLED);
// return reactRootView;
// }
// }
}
4 changes: 1 addition & 3 deletions ci/Jenkinsfile.ios
Original file line number Diff line number Diff line change
Expand Up @@ -55,9 +55,7 @@ pipeline {
}
stage('JSBundle') {
steps {
script {
ios.jsbundle()
}
script { ios.jsbundle() }
}
}
stage('Bundle') {
Expand Down
20 changes: 10 additions & 10 deletions ios/Podfile
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ require_relative '../node_modules/react-native/scripts/react_native_pods'
require_relative '../node_modules/@react-native-community/cli-platform-ios/native_modules'

platform :ios, '12.4'
# install! 'cocoapods', :deterministic_uuids => false

production = ENV["PRODUCTION"] == "1"

Expand All @@ -20,15 +19,16 @@ abstract_target 'Status' do
# Flags change depending on the env values.
flags = get_default_flags()

use_react_native!(:path => config[:reactNativePath],
# to enable hermes on iOS, change `false` to `true` and then install pods
:production => production,
:hermes_enabled => flags[:hermes_enabled],
:fabric_enabled => flags[:fabric_enabled],
:use_codegen_discovery => true,
# :flipper_configuration => FlipperConfiguration.enabled,
# An absolute path to your application root.
:app_path => "#{Pod::Config.instance.installation_root}/.."
use_react_native!(
:path => config[:reactNativePath],
# to enable hermes on iOS, change `false` to `true` and then install pods
:production => production,
:hermes_enabled => flags[:hermes_enabled],
:fabric_enabled => flags[:fabric_enabled],
:use_codegen_discovery => true,
# :flipper_configuration => FlipperConfiguration.enabled,
# An absolute path to your application root.
:app_path => "#{Pod::Config.instance.installation_root}/.."
)

pod 'react-native-image-resizer', :path => '../node_modules/react-native-image-resizer'
Expand Down
2 changes: 1 addition & 1 deletion nix/deps/gradle/gradle_parser.awk
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ function findPackage(line, regex) {
# Lines after configuration name list packages
for (getline line; line != ""; getline line) {
# React Native is provided by node_modules
if (line ~ "com.facebook.react:") { continue }
if (line ~ "com.facebook.react:(react-native|hermes-engine)") { continue }

# Example: +--- org.jetbrains.kotlin:kotlin-stdlib:1.3.50
if (findPackage(line, "--- ([^ :]+):([^ :]+):([^ :]+)$")) {
Expand Down

0 comments on commit 8e99442

Please sign in to comment.