Skip to content

reduce logging#2

Open
tomjnaim wants to merge 36 commits intoMonikerSonic:masterfrom
naimaudio:master
Open

reduce logging#2
tomjnaim wants to merge 36 commits intoMonikerSonic:masterfrom
naimaudio:master

Conversation

@tomjnaim
Copy link

While trying to debug something using the iOS simulator I found the default level of logging has crept up a lot recently. I have done the following:

  1. Reduced the default level of logging for modules that have log level control
  2. Deleted log statements that have no context or are obviously left over from debug sessions in the distant past
  3. Commented out debug that looks like it has a purpose but has little contextual info in the log. If you revisit this module and want to re-enable it do so in a conditional manner so a given module can have it's debug enabled/disabled
  4. Added some context to debug messages that are flagging errors but had little contextual info in the log message

nickhingston and others added 30 commits April 20, 2018 16:20
…ource

APP-2737 remove primary event source
…ource

APP-2737 ssekit remove primary source
Reduced amount of default logging
APP-4585 fix xcode 12 compiler warnings
…SSEKit

APP-5893: SSEKit logging working.
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.

4 participants