Skip to content
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

Update README.md #65

Merged
merged 4 commits into from
Aug 24, 2016
Merged

Update README.md #65

merged 4 commits into from
Aug 24, 2016

Conversation

lashkhi
Copy link
Contributor

@lashkhi lashkhi commented Aug 2, 2016

Added description on how to run example projects without Carthage

Added description on how to run example projects without Carthage
@lashkhi
Copy link
Contributor Author

lashkhi commented Aug 2, 2016

@jbrophy17 check this one please🙂

@jbrophy17
Copy link
Contributor

Thanks for the PR, just need to verify the steps are accurate and then I'll merge this in :)


For Objective-C projects, set the **Embedded Content Contains Swift Code** flag in your project to **Yes** (found under **Build Options** in the **Build Settings** tab).

Open .xcworkspace and navigate to **General** tab, scroll to **Embedded Binaries** select ObjectMapper.framework and click the `-` button, do the same for UberRides.framework. Now go to **Build Settings** tab and scroll to **Search Paths**, click on *Framework Search Paths** and remove the line $(PROJECT_DIR)/Carthage/Build/iOS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Framework Search Paths* is missing an asterisk, should be Framework Search Paths

@jbrophy17
Copy link
Contributor

@lashkhi sorry for the delay! Left a few comments, once we clean these up I can go ahead and merge it

Text clean up: removed  copy & paste issues, added more attention to removing Carthage
@lashkhi
Copy link
Contributor Author

lashkhi commented Aug 10, 2016

@jbrophy17 Hi, no problem for delay! Thanks for reviewing it— I've made changes you commented on.


For Objective-C projects, set the **Embedded Content Contains Swift Code** flag in your project to **Yes** (found under **Build Options** in the **Build Settings** tab).

Open .xcworkspace and navigate to **General** tab, scroll to **Embedded Binaries** select ObjectMapper.framework and click the `-` button, do the same for UberRides.framework. Now go to **Build Settings** tab and scroll to **Search Paths**, click on **Framework Search Paths** and remove the line $(PROJECT_DIR)/Carthage/Build/iOS.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also think it makes sense to move this section right after steps to remove dependencies. That way, removing all the carthage stuff comes first, and then the Cocoapods steps

@jbrophy17
Copy link
Contributor

just a few more comments, otherwise it is looking good

Put the section for removing Carthage together, some text changes for better understanding
@lashkhi
Copy link
Contributor Author

lashkhi commented Aug 11, 2016

@jbrophy17 Yeah, that makes sense. Updated 🙂

@@ -930,9 +930,11 @@ carthage update --platform iOS
This will build the required dependencies. Once you do that, open `Swift SDK.xcodeproj` or `Obj-C SDK.xcodeproj` in Xcode and run it.

### CocoaPods
First, you will have to remove Carthage dependencies. Navigate to `examples/Swift SDK` or `examples/Obj-C SDK` and remove `Cartfile` and `Cartfile.resolved`. If you see a `Carthage` folder, remove that as well.
First, you will have to remove Carthage dependencies. Navigate to `examples/Swift SDK` or `examples/Obj-C SDK` and remove `Cartfile` and `Cartfile.resolved`. If you see a `Carthage` folder, remove that as well. Open .xcworkspace and navigate to **General** tab, scroll to **Embedded Binaries** select `ObjectMapper.framework` and click the `-` button, do the same for `UberRides.framework`. Now go to **Build Settings** tab and scroll to **Search Paths**, click on **Framework Search Paths** and remove the line $(PROJECT_DIR)/Carthage/Build/iOS.
Now got to **Build Phases** find the **Copy Carthage Frameworks** and remove it.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor typo here, "got to" instead of "go to"

@jbrophy17
Copy link
Contributor

Hey @lashkhi sorry for the delay here, just one minor typo and then I will merge this in

@lashkhi
Copy link
Contributor Author

lashkhi commented Aug 23, 2016

@jbrophy17 Oh, sorry for this mistake— fixed it already

@jbrophy17
Copy link
Contributor

Awesome, looks good

@jbrophy17 jbrophy17 merged commit 2dacc7f into uber:0.5.x-dev Aug 24, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants