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

reflect breaking changes in fatih/color:85fc70c (color.Set() private) #187

Closed
wants to merge 1 commit into from

Conversation

bjaglin
Copy link
Collaborator

@bjaglin bjaglin commented May 4, 2015

Simply allows to build Crane against latest fatih/color - no functional change.
Fixes #181 ☀️

@fatih
Copy link

fatih commented May 4, 2015

Sorry again, this is imho something I did. I've totally forget about the publicity of color.Set(). I've reverted it again with fatih/color#14. Please let me know if the there is something to do.

@bjaglin
Copy link
Collaborator Author

bjaglin commented May 4, 2015

OK, maybe let's hang on with this one - the breaking change is apparently going to be reverted (see issue above).

@bjaglin bjaglin closed this May 4, 2015
@michaelsauter
Copy link
Owner

@fatih thanks for reverting!

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.

Set up CI hooks
4 participants