Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

HelloSora の参照する React Native を v0.62.2 にバージョン・アップする #16

Merged
merged 12 commits into from
Jul 4, 2020

Conversation

enm10k
Copy link
Contributor

@enm10k enm10k commented Jun 28, 2020

変更内容

  • HelloSora が参照する React Native を v0.62.2 にバージョン・アップしました

作業メモ

React Native v0.62.2 で作成されたプロジェクトと HelloSora のファイルの比較

v0.62.2 で作成するプロジェクトの名前を揃えたかったので、 tmp ディレクトリを掘ってその中で react-native init しています

$ mkdir tmp && cd tmp && npx react-native@0.62.2 init HelloSora && cd ..

プロジェクトのディレクトリの構成を比較 ... 変化なし

$ ls -dl tmp/HelloSora/*/ | grep -v node_modules
drwxr-xr-x    3 enm10k  staff     96  6 27 19:20 tmp/HelloSora/__tests__/
drwxr-xr-x    9 enm10k  staff    288  6 27 19:20 tmp/HelloSora/android/
drwxr-xr-x   11 enm10k  staff    352  6 27 19:20 tmp/HelloSora/ios/

$ ls -dl HelloSora/*/ | grep -v node_modules
drwxr-xr-x   3 enm10k  staff   96  6 23 23:59 HelloSora/__tests__/
drwxr-xr-x   9 enm10k  staff  288  6 23 23:59 HelloSora/android/
drwxr-xr-x  10 enm10k  staff  320  6 23 23:59 HelloSora/ios/

比較

# プロジェクトのルート・ディレクトリの差分
$ diff -q HelloSora/ tmp/HelloSora/ | grep -iv "common subdirectories"
Files HelloSora/.flowconfig and tmp/HelloSora/.flowconfig differ
Files HelloSora/.gitignore and tmp/HelloSora/.gitignore differ
Only in tmp/HelloSora/: .prettierrc.js
Files HelloSora/App.js and tmp/HelloSora/App.js differ
Only in HelloSora/: README.md
Only in HelloSora/: Sora.js
Files HelloSora/app.json and tmp/HelloSora/app.json differ
Only in HelloSora/: clean_build.sh
Files HelloSora/index.js and tmp/HelloSora/index.js differ
Only in tmp/HelloSora/: node_modules
Files HelloSora/package.json and tmp/HelloSora/package.json differ
Files HelloSora/yarn.lock and tmp/HelloSora/yarn.lock differ

# __tests__ ディレクトリの差分 ... なし
$ diff HelloSora/__tests__ tmp/HelloSora/__tests__ | wc -l
       0

# android ディレクトリ以下の差分
$ diff -qr HelloSora/android tmp/HelloSora/android
Only in HelloSora/android/app: BUCK
Only in tmp/HelloSora/android/app: _BUCK
Files HelloSora/android/app/build.gradle and tmp/HelloSora/android/app/build.gradle differ
Only in tmp/HelloSora/android/app: debug.keystore
Only in tmp/HelloSora/android/app/src/debug: java
Files HelloSora/android/app/src/main/AndroidManifest.xml and tmp/HelloSora/android/app/src/main/AndroidManifest.xml differ
Only in HelloSora/android/app/src/main: assets
Files HelloSora/android/app/src/main/java/com/hellosora/MainActivity.java and tmp/HelloSora/android/app/src/main/java/com/hellosora/MainActivity.java differ
Files HelloSora/android/app/src/main/java/com/hellosora/MainApplication.java and tmp/HelloSora/android/app/src/main/java/com/hellosora/MainApplication.java differ
Files HelloSora/android/build.gradle and tmp/HelloSora/android/build.gradle differ
Files HelloSora/android/gradle/wrapper/gradle-wrapper.properties and tmp/HelloSora/android/gradle/wrapper/gradle-wrapper.properties differ
Files HelloSora/android/gradle.properties and tmp/HelloSora/android/gradle.properties differ
Files HelloSora/android/gradlew and tmp/HelloSora/android/gradlew differ
Files HelloSora/android/gradlew.bat and tmp/HelloSora/android/gradlew.bat differ

# ios ディレクトリ以下の差分
$ diff -qr HelloSora/ios tmp/HelloSora/ios
Files HelloSora/ios/HelloSora/AppDelegate.h and tmp/HelloSora/ios/HelloSora/AppDelegate.h differ
Files HelloSora/ios/HelloSora/AppDelegate.m and tmp/HelloSora/ios/HelloSora/AppDelegate.m differ
Files HelloSora/ios/HelloSora/Info.plist and tmp/HelloSora/ios/HelloSora/Info.plist differ
Files HelloSora/ios/HelloSora/main.m and tmp/HelloSora/ios/HelloSora/main.m differ
Files HelloSora/ios/HelloSora.xcodeproj/project.pbxproj and tmp/HelloSora/ios/HelloSora.xcodeproj/project.pbxproj differ
Files HelloSora/ios/HelloSora.xcodeproj/xcshareddata/xcschemes/HelloSora-tvOS.xcscheme and tmp/HelloSora/ios/HelloSora.xcodeproj/xcshareddata/xcschemes/HelloSora-tvOS.xcscheme differ
Files HelloSora/ios/HelloSora.xcodeproj/xcshareddata/xcschemes/HelloSora.xcscheme and tmp/HelloSora/ios/HelloSora.xcodeproj/xcshareddata/xcschemes/HelloSora.xcscheme differ
Files HelloSora/ios/HelloSoraTests/HelloSoraTests.m and tmp/HelloSora/ios/HelloSoraTests/HelloSoraTests.m differ
Files HelloSora/ios/Podfile and tmp/HelloSora/ios/Podfile differ
Files HelloSora/ios/Podfile.lock and tmp/HelloSora/ios/Podfile.lock differ
Only in tmp/HelloSora/ios: Pods

iOS のビルド・エラーを修正するために参照したイシュー

iOS のビルド時に、 React Native に v0.62 から導入された Flipper 関連でビルド・エラーが発生しました
修正のために以下のイシューを参照しました

@enm10k enm10k changed the title WIP: HelloSora の参照する React Native を v0.62.2 にバージョン・アップする HelloSora の参照する React Native を v0.62.2 にバージョン・アップする Jun 28, 2020
@enm10k enm10k requested a review from kdxu June 28, 2020 10:20
@kdxu
Copy link
Contributor

kdxu commented Jun 30, 2020

最新に保つという意味で ./HelloSora/android 以下の差分をコミットしても良さそうです。

@kdxu
Copy link
Contributor

kdxu commented Jun 30, 2020

見ました。また、iOS でビルドできることを確認しました。

@enm10k
Copy link
Contributor Author

enm10k commented Jul 1, 2020

最新に保つという意味で ./HelloSora/android 以下の差分をコミットしても良さそうです。

fc3302c で対応しました

@enm10k
Copy link
Contributor Author

enm10k commented Jul 1, 2020

@kdxu レビューありがとうございます!
修正しましたので、確認をお願いいたします

@kdxu
Copy link
Contributor

kdxu commented Jul 1, 2020

@enm10k
ありがとうございます。
./HelloSora/android 以下の差分がコミットされていることを確認しました。
こちらのコメント を再度確認していただければと思います。

@enm10k
Copy link
Contributor Author

enm10k commented Jul 2, 2020

@kdxu 修正しました
再度、確認をお願いいたします

@kdxu
Copy link
Contributor

kdxu commented Jul 3, 2020

LGTM

@enm10k
Copy link
Contributor Author

enm10k commented Jul 4, 2020

ありがとうございます
マージします

@enm10k enm10k merged commit 0fdb025 into develop Jul 4, 2020
@enm10k enm10k deleted the feature/upgrade-hellosora-rn branch July 4, 2020 14:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants