-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Mobile Release v1.59.0 #33913
Mobile Release v1.59.0 #33913
Conversation
Size Change: 0 B Total Size: 1.07 MB ℹ️ View Unchanged
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good so far. I left one comment regarding an odd change to the Demo app Xcode configuration.
@@ -667,7 +667,7 @@ | |||
"CODE_SIGN_IDENTITY[sdk=iphoneos*]" = "iPhone Developer"; | |||
COPY_PHASE_STRIP = NO; | |||
ENABLE_STRICT_OBJC_MSGSEND = YES; | |||
"EXCLUDED_ARCHS[sdk=iphonesimulator*]" = "arm64 "; | |||
"EXCLUDED_ARCHS[sdk=iphonesimulator*]" = ""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems like an odd change. Based upon e980666, it would appear this was performed by the release script. Any ideas as to why this occurred?
Given its a part of the Demo app and references simulators, I would imagine it would have no negative impact on the release build.
Description
Release 1.59.0 of the react-native-editor and Gutenberg-Mobile.
For more information about this release and testing instructions, please see the related Gutenberg-Mobile PR: wordpress-mobile/gutenberg-mobile#3805
Checklist:
*.native.js
files for terms that need renaming or removal).