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

Add start/stop command gemstash outputs #58

Merged
merged 2 commits into from
Dec 7, 2015

Conversation

farukaydin
Copy link
Member

Fixes #56

I didn't add any test since I think it doesn't need.

@pcarranza
Copy link
Contributor

Just wonder if it makes more sense to do it with cli.say rather than the logger. Then you can even say it in green 😄

@smellsblue thoughts?

@farukaydin
Copy link
Member Author

@pcarranza updated like you mentioned. I think it's better (and green 😄)

@pcarranza
Copy link
Contributor

LFTM 👍

Let's wait for @smellsblue if you don't mind

@smellsblue
Copy link
Contributor

👍

smellsblue added a commit that referenced this pull request Dec 7, 2015
Add start/stop command gemstash outputs
@smellsblue smellsblue merged commit a643bce into rubygems:master Dec 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants