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

Added ability to change log level - default level set to Error #448

Merged
merged 2 commits into from
Jul 10, 2024

Conversation

thatcoderguy
Copy link
Contributor

@thatcoderguy thatcoderguy commented Jul 1, 2024

  • Removed the "enable debug" option from config
  • We now have 5 different logging levels which can be set
  • Logging enabled by default, but set to Error (so nothing only exceptions are logged)
  • The next PR will be me adding different more logging statements, but with the relevant logging level

@thatcoderguy
Copy link
Contributor Author

thatcoderguy commented Jul 1, 2024

Not a fan of where I've moved config.json check to in the code... another way of doing would be to set the default logging level to Debug, then just before the config file is read, change it to Error, then change it again when we know what the config file says.

@sim0n00ps sim0n00ps merged commit 6ae9750 into sim0n00ps:master Jul 10, 2024
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.

2 participants