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

reduce logging #2

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

reduce logging #2

wants to merge 36 commits into from

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