-
Notifications
You must be signed in to change notification settings - Fork 24.5k
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
Add xcscheme files back in the template #28198
Conversation
Can you make the PR title clearer? I’d edit it myself but I can’t figure out how to do that from mobile... |
Done, I missed the title when submitting 🤦♂ |
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.
@TheSavior has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was successfully merged by @pvinis in a715dec. When will my fix make it into a release? | Upcoming Releases |
Summary: Adding `xcscheme` files for iOS template back in. They were removed in #26471 after #25451. We have reverted the change on `.gitignore`, as this directory is supposed to be in git, but we didn't revert the removed files. Now they are back as well. You can also see here that the two files are still missing, they got removed on rc.0 and are still missing on rc.3. https://react-native-community.github.io/upgrade-helper/?from=0.61.5&to=0.62.0-rc.3 ## Changelog [iOS] [Changed] - Add `xcscheme` files for iOS template back in. Pull Request resolved: #28198 Test Plan: `init` a nre project, and check if the files are there or not. The paths are `ios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp.xcscheme` and `ios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp-tvOS.xcscheme` Reviewed By: cpojer Differential Revision: D20179447 Pulled By: TheSavior fbshipit-source-id: b0f08c0f32d6bb7630179bc2fe46d9ac10f6c1d7
Summary: Adding `xcscheme` files for iOS template back in. They were removed in facebook#26471 after facebook#25451. We have reverted the change on `.gitignore`, as this directory is supposed to be in git, but we didn't revert the removed files. Now they are back as well. You can also see here that the two files are still missing, they got removed on rc.0 and are still missing on rc.3. https://react-native-community.github.io/upgrade-helper/?from=0.61.5&to=0.62.0-rc.3 ## Changelog [iOS] [Changed] - Add `xcscheme` files for iOS template back in. Pull Request resolved: facebook#28198 Test Plan: `init` a nre project, and check if the files are there or not. The paths are `ios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp.xcscheme` and `ios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp-tvOS.xcscheme` Reviewed By: cpojer Differential Revision: D20179447 Pulled By: TheSavior fbshipit-source-id: b0f08c0f32d6bb7630179bc2fe46d9ac10f6c1d7
Summary
Adding
xcscheme
files for iOS template back in. They were removed in #26471 after #25451. We have reverted the change on.gitignore
, as this directory is supposed to be in git, but we didn't revert the removed files. Now they are back as well.You can also see here that the two files are still missing, they got removed on rc.0 and are still missing on rc.3.
https://react-native-community.github.io/upgrade-helper/?from=0.61.5&to=0.62.0-rc.3
Changelog
[iOS] [Changed] - Add
xcscheme
files for iOS template back in.Test Plan
init
a nre project, and check if the files are there or not. The paths areios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp.xcscheme
andios/MyApp.xcodeproj/xcshareddata/xcschemes/MyApp-tvOS.xcscheme