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

Bring back doFmtVerbLevelColor on Windows #76

Open
wants to merge 30 commits into
base: master
Choose a base branch
from

Conversation

zanderz
Copy link
Contributor

@zanderz zanderz commented Dec 7, 2015

This moves several things back to a shared log.go. We have started supporting ansi color control codes on Windows by (using an io.writer that looks for them)[https://github.com/keybase/client/blob/master/go/libkb/output_windows.go]. That could also be brought over to op/go-logging if anyone wants.

This is merged on top of your recent Windows fix - thanks again.

Steve Sanders and others added 11 commits December 6, 2015 17:38
Fixes this error when vendoring go/logger:

github.com/keybase/client/go/logger/standard.go:208: log.internal.Criticalf undefined (type *logging.Logger has no field or method Criticalf)
Fixes missing Criticalf method
Revert "Fixes missing Criticalf method"
Copy link

@nim-nim nim-nim left a comment

Choose a reason for hiding this comment

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

Needed for github.com/hudl/fargo too

@zanderz
Copy link
Contributor Author

zanderz commented Oct 25, 2017

Someone else will need to merge, I lack permission

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.

7 participants