Skip to content
This repository has been archived by the owner on Jul 2, 2018. It is now read-only.

Use colored2 instead of colorize for CLI colors #66

Merged
merged 1 commit into from
Feb 15, 2017

Conversation

k0nserv
Copy link
Contributor

@k0nserv k0nserv commented Feb 14, 2017

Colorize and colored2 cannot be used at the same time due to
them defining conflicting entities in the global space. Since
CocoaPods uses colored2 and it's likely that someone would use
CocoaSeeds along side CocoaPods it's better to use colored2 in
CocoaSeeds too.

Colorize and colored2 cannot be used at the same time due to
them defining conflicting entities in the global space. Since
CocoaPods uses colored2 and it's likely that someone would use
CocoaSeeds along side CocoaPods it's better to use colored2 in
CocoaSeeds too.
gemspec.scan(/(?<=s\.add_runtime_dependency ).*/).each do |dependency|
eval "gem #{dependency}"
end
gemspec
Copy link
Owner

Choose a reason for hiding this comment

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

Can you explain me about this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It's the recommended way to include run time and delelopment dependecies for local development of the gem.

From the bundler docs

Gemfile: Used to manage gem dependencies for our library’s development. This file contains a gemspec line meaning that Bundler will include dependencies specified in foodie.gemspec too. It’s best practice to specify all the gems that our library depends on in the gemspec.

I also added rake as a development dependency since it's used to run the tests

Copy link
Owner

Choose a reason for hiding this comment

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

Wow didn't know that. Thanks for your detailed explanation!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No problem 👌

@devxoul devxoul merged commit 1f243db into devxoul:master Feb 15, 2017
@devxoul
Copy link
Owner

devxoul commented Feb 15, 2017

@k0nserv, thanks for your contribution! Your commits have been shipped along with 0.8.3 release 👍

@k0nserv k0nserv deleted the replace-colorize branch February 15, 2017 17:48
@k0nserv
Copy link
Contributor Author

k0nserv commented Feb 15, 2017

Wow! Awesome thanks for the speedy release 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants