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.

Sign up for GitHub

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

Jump to bottom

Convert logger into a class. #22129

Merged

Koenkk merged 1 commit into Koenkk:dev from Nerivec:logger-class

Apr 7, 2024

Collaborator

Nerivec commented Apr 7, 2024

Additional changes:

Short-circuit debug calls to winston if log_level isn't debug (improves performance, since all levels are synced across transports it causes no issue).
Remove init/logOutput calls from Controller (& tests), no longer needed.
Fix up first logged line that displays what logging is active/enabled. Purely cosmetic.


          Convert logger into a class.

d84e455

Nerivec force-pushed the logger-class branch from 16f24b0 to d84e455 Compare

April 7, 2024 18:51

Koenkk merged commit 3c1a4af into Koenkk:dev

8 checks passed

Owner

Koenkk commented Apr 7, 2024

Thanks!

Koenkk mentioned this pull request

chore(dev): release 1.37.0 #22022

Merged

Nerivec deleted the logger-class branch

April 7, 2024 19:05

Nerivec mentioned this pull request

feat(logs): user-selectable logs limit nurikk/zigbee2mqtt-frontend#1999

Merged

Nerivec mentioned this pull request

Fix logger crashing on restart #22565

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet