/juju

Add colours to debug-log. #6025

Merged
merged 2 commits into from Aug 18, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@howbazaar @mjs @jujubot
@howbazaar
Owner

howbazaar commented Aug 18, 2016

Show logging in glorious colour.
cmd/juju/commands/debuglog.go
}
return nil
}
-func (c *debugLogCommand) writeLogRecord(w io.Writer, r api.LogMessage) {
+var SeverityColor = map[string]*ansiterm.Context{
@mjs

mjs Aug 18, 2016

Contributor

Is this the same as what's in loggo? Are you duplicating to decouple from the default in loggo?

@howbazaar

howbazaar Aug 18, 2016

Owner

Due to the api sending strings back for levels.

@mjs
Contributor

mjs commented Aug 18, 2016

LGTM
@howbazaar
Owner

howbazaar commented Aug 18, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Aug 18, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit dbc0521 into juju:master Aug 18, 2016

@howbazaar howbazaar deleted the howbazaar:color-debug-log branch Aug 18, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment