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

Correct the spelling of CocoaPods in README #2

Merged
merged 1 commit into from May 18, 2016
Merged

Correct the spelling of CocoaPods in README #2

merged 1 commit into from May 18, 2016

Conversation

ReadmeCritic
Copy link
Contributor

This pull request corrects the spelling of CocoaPods 🤓
https://github.com/CocoaPods/shared_resources/tree/master/media

One day I’ll make a bot that looks through the READMEs of all Pods, looks to see if it uses “Cocoapods” and PRs “CocoaPods” :D

— Ørta (@orta) February 10, 2016
<script async src="//platform.twitter.com/widgets.js" charset="utf-8"></script>

Created with cocoapods-readme.

@ReadmeCritic
Copy link
Contributor Author

Hey you know, the instructions should use pod update instead of pod install.. let me know if you would like me to correct that as well https://guides.cocoapods.org/using/pod-install-vs-update.html

@orta
Copy link

orta commented May 18, 2016

Assuming you're referring to:

Then, run the following command:

$ pod install

You don't want to change that 👍 - that's correct

@ReadmeCritic
Copy link
Contributor Author

😕

@ReadmeCritic
Copy link
Contributor Author

^ confused

@hectormatos2011
Copy link
Contributor

When bringing in a CocoaPod for the first time in your project, you want to run pod install for first time installation. This command keeps the current pods already resolved in the lock file intact and just installs the new Pod (in this case, PrediKit). pod update will run the dependency resolver and update every pod as well as installing the new pods referenced in the Podfile.

@hectormatos2011
Copy link
Contributor

Thanks for the correction though! I always make this mistake and remember to change it later. Merging.

@hectormatos2011 hectormatos2011 merged commit 977347a into KrakenDev:master May 18, 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.

None yet

3 participants