-
Notifications
You must be signed in to change notification settings - Fork 24.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
remove xcshareddata
from gitignore
#27563
Conversation
this is meant to be shared
Looks like this was added by @karanjthakkar in #25451 |
Ah, good spotting. I remember writing about this, but I couldn't find it. Anyway, like I said there, |
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 is landing 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 7980615. When will my fix make it into a release? | Upcoming Releases |
Summary: Files that are inside `xcshareddata` are meant to be "shared", and that means committed to git. These files are for example `ios/RnDiffApp.xcodeproj/xcshareddata/xcschemes/RnDiffApp.xcscheme`, and if ignored that means they are removed from the repo or are missing when cloning. These should be present. They are the shared schemes so Xcode knows what and how to build, on everyone's machine. I noticed it being there in a new project on 0.62.0-rc.0. ## Changelog [iOS] [Changed] - Remove the xcshareddata from .gitignore Pull Request resolved: #27563 Differential Revision: D19166364 Pulled By: TheSavior fbshipit-source-id: 0377dc5b08ba04d121f9ed5e4027942a7a9f463f
Summary
Files that are inside
xcshareddata
are meant to be "shared", and that means committed to git. These files are for exampleios/RnDiffApp.xcodeproj/xcshareddata/xcschemes/RnDiffApp.xcscheme
, and if ignored that means they are removed from the repo or are missing when cloning. These should be present. They are the shared schemes so Xcode knows what and how to build, on everyone's machine.I noticed it being there in a new project on 0.62.0-rc.0.
Changelog
[iOS] [Changed] - Remove the xcshareddata from .gitignore
Test Plan