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

GlobalLoggerGuard should implement Debug #246

Open
Diggsey opened this issue Nov 22, 2019 · 1 comment
Open

GlobalLoggerGuard should implement Debug #246

Diggsey opened this issue Nov 22, 2019 · 1 comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. P-low Priority: Low

Comments

@Diggsey
Copy link

Diggsey commented Nov 22, 2019

All public types should implement Debug unless there's a good reason not to.

Since the guard removes the logger when it is dropped, it can be necessary to store the guard as part of a struct which may have Debug bounds.

@dpc
Copy link
Collaborator

dpc commented Nov 22, 2019

PR, pls. Thx!

@Techcable Techcable added C-enhancement Category: An issue proposing an enhancement or a PR with one. P-low Priority: Low labels Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: An issue proposing an enhancement or a PR with one. P-low Priority: Low
Projects
None yet
Development

No branches or pull requests

3 participants