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 per logger colors #1005

Merged
merged 3 commits into from Feb 25, 2015

Conversation

2 participants
@adamcik
Member

adamcik commented Feb 25, 2015

No description provided.

@@ -128,6 +128,14 @@ Logging configuration
The ``loglevels`` config section can be used to change the log level for
specific parts of Mopidy during development or debugging. Each key in the
config section should match the name of a logger. The value is the log
level to use for that logger, one of ``black``, ``red``, ``green``,
``yellow``, ``blue``, ``magenta``, ``cyan`` or ``white``.

This comment has been minimized.

@jodal

jodal Feb 25, 2015

Member

You've mixed up the list of log levels and the list of log colors.

class LogLevelConfigSchema(object):
"""Special cased schema for handling a config section with loglevels.
class MapConfigSchema(object):
"""Special cased schema for handling mulitple keys with the same type.

This comment has been minimized.

@jodal

jodal Feb 25, 2015

Member

s/mulitple/multiple/

This comment has been minimized.

@jodal

jodal Feb 25, 2015

Member

Maybe not very "Special cased" any more?

@jodal jodal added this to the v0.20 - Audio cleanup 1 milestone Feb 25, 2015

@jodal jodal self-assigned this Feb 25, 2015

@jodal

This comment has been minimized.

Member

jodal commented Feb 25, 2015

+1 with comments

@adamcik

This comment has been minimized.

Member

adamcik commented Feb 25, 2015

Fixed.

jodal added a commit that referenced this pull request Feb 25, 2015

Merge pull request #1005 from adamcik/fix/808
Add per logger colors

@jodal jodal merged commit 5536c79 into mopidy:develop Feb 25, 2015

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Scrutinizer 4 updated code elements
Details

@jodal jodal referenced this pull request Feb 25, 2015

Closed

Add per logger colors? #808

@adamcik adamcik deleted the adamcik:fix/808 branch Mar 2, 2015

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