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

Util+Everywhere: Logging & Logging Managment #709

Closed
wants to merge 3 commits into from

Conversation

filiphsps
Copy link
Contributor

@filiphsps filiphsps commented Nov 6, 2022

My goal with this PR is to make the debugging of BuggieBot a little bit more pleasant; and hopefully find a way to properly catch #533.

Configuration required:

  • log_level: number based on the LogLevel enum (optional, currently defaults to Verbose).
  • log_channel_id: the logger's output channel (technically optional, but no).
  • privileged_group_ids: comma separated list of group ids (optional, defaults to 100 Commit Club)

A new logging channel should probably be configured in such a way that notifications are turned off by default as to not spam everybody.

image

@filiphsps filiphsps marked this pull request as ready for review November 6, 2022 19:57
@filiphsps filiphsps force-pushed the dev-logger branch 3 times, most recently from d5eaa66 to fdb763d Compare November 6, 2022 20:55
@filiphsps filiphsps marked this pull request as draft November 7, 2022 04:08
@filiphsps filiphsps force-pushed the dev-logger branch 5 times, most recently from f17d0cc to 13e2156 Compare November 7, 2022 04:57
Configuration:
  - `log_level`: number based on the `LogLevel` enum.
  - `log_channel_id`: the logger's output channel.
These commands can be used by a privileged member for debugging
purposes.
@filiphsps
Copy link
Contributor Author

Gonne rethink how to handle this in the best way.

@filiphsps filiphsps closed this Nov 10, 2022
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.

1 participant