-
Notifications
You must be signed in to change notification settings - Fork 500
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
Tools: Add a script to initialize quickly and easily the project #4596
Merged
Merged
Changes from 4 commits
Commits
Show all changes
9 commits
Select commit
Hold shift + click to select a range
3baf66d
Add a script to initialize quickly and easily the project.
SBiOSoftWhare 0971654
setup_project.sh: Update comments.
SBiOSoftWhare e77f788
Update changes
SBiOSoftWhare 4ad047a
Update Readme
SBiOSoftWhare 8484fcb
Revert "Update Readme"
SBiOSoftWhare 37ac6aa
Update install guide.
SBiOSoftWhare 46db3aa
Merge branch 'develop' into steve/add_setup_project_script
SBiOSoftWhare d9cad24
Update INSTALL.md
SBiOSoftWhare 2d3a912
Merge develop into steve/add_setup_project_script
SBiOSoftWhare File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,30 @@ | ||
#!/bin/bash | ||
|
||
# Use this script to setup the Xcode project | ||
|
||
# Remove existing project file if any | ||
rm -r Riot.xcodeproj | ||
|
||
# Create the xcodeproj with all project source files | ||
xcodegen | ||
|
||
# Use appropriated dependencies | ||
|
||
# Check if Podfile changed in unstaged | ||
git diff --exit-code --quiet --name-only Podfile | ||
PODFILE_HAS_CHANGED_UNSTAGED=$? | ||
# Check if Podfile changed in staged | ||
git diff --staged --exit-code --quiet --name-only Podfile | ||
PODFILE_HAS_CHANGED_STAGED=$? | ||
|
||
# If Podfile has changed locally do not modify it | ||
# otherwise use the appropriated dependencies according to the current branch | ||
if [[ "$PODFILE_HAS_CHANGED_UNSTAGED" -eq 1 || "$PODFILE_HAS_CHANGED_STAGED" -eq 1 ]]; then | ||
echo "Podfile has been changed locally do not modify it" | ||
else | ||
echo "Podfile has not been changed locally, use appropriated dependencies according to the current branch" | ||
bundle exec fastlane point_dependencies_to_same_feature | ||
fi | ||
|
||
# Create the xcworkspace with all project dependencies | ||
pod install |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I would prefer to keep using the "xcodegen / pod install" combo here for simplicity and for easy understanding of what is happening.
I never needed to use more complex setup. You could mention setup_project.sh in INSTALL.md for advanced usage. While possible, I prefer to keep thing simple at first place.