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

Move to CocoaPods 1.2.1 #300

Merged

Conversation

garriguv
Copy link
Contributor

@garriguv garriguv commented May 12, 2017

bundle install failed on my machine because I use ruby 2.4.1 and the
json gem version 1.8.3 was not compatible.

This problem has now been fixed:
ruby/json#303

While we're at it, why not update all the gems 😃

Since we're updating the gems, let's move to the latest CocoaPods as well.

bundle install failed on my machine because I use ruby 2.4.1 and the
json gem version 1.8.3 was not compatible.

This problem has now been fixed:
ruby/json#303
It is needed for the cocoapods-update-if-you-dare gem...
@garriguv
Copy link
Contributor Author

The Cocoapods update triggers a lot of changes to the project file after running bundle exec pod install.

I'm looking at them and may push the changes to this branch, or create a new pull-request.

@garriguv garriguv changed the title Update Ruby dependencies Move to CocoaPods 1.2.1 May 12, 2017
@SourceryBot
Copy link

SourceryBot commented May 12, 2017

1 Warning
⚠️ Big PR

Generated by 🚫 Danger

Gemfile Outdated
@@ -2,6 +2,7 @@ source 'https://rubygems.org'

gem 'cocoapods'
gem 'cocoapods-update-if-you-dare', '~> 0.1.1'
gem 'colored'
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure if we need a new dependency here

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We actually do, since cocoapods-update-if-you-dare does not specify this dependency explicitly. And cocoapods moved to colored2: CocoaPods/Xcodeproj#463

I opened a PR on the project itself to fix it: Ashton-W/cocoapods-update-if-you-dare#2

Copy link
Collaborator

Choose a reason for hiding this comment

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

Ah, thanks, @garriguv, I've missed the commit description 😄 And thanks for the PR to the CocoaPods plugin 👍 Maybe the @Ashton-W can merge it soon and release a new version, so we again won't require the colored dependency specified in the gem.

Copy link
Collaborator

Choose a reason for hiding this comment

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

But if it's not released soon, the PR looks cool and we could merge it as is. Does it sound fine for you @garriguv ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah that sounds good. I'll open another PR to remove colored once Ashton-W/cocoapods-update-if-you-dare#2 is merged and a new version of the gem is released.

Choose a reason for hiding this comment

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

I just released 0.2.0 for you :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

@garriguv I think there is no need for another PR. Just remove colored here from this one and update bundle once again and we're good to go 🚀

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done 🚀

Copy link
Collaborator

Choose a reason for hiding this comment

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

Amazing, congratulations on your first contribution 🎉

I dared :)

Also got rid of the colored dependency.
@Antondomashnev Antondomashnev merged commit da3394a into krzysztofzablocki:master May 13, 2017
@garriguv garriguv deleted the garriguv/bundle-update branch May 14, 2017 08:01
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.

4 participants