-
Notifications
You must be signed in to change notification settings - Fork 194
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
fix: bunyan logs rotating file are broken, don't use them #3145
Conversation
d156756
to
03df7ba
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on my machine and it seems to be working fine. Delete logs folder and it correctly remakes it.
The test will be in the next release that other's logging has been fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚢
use instead of showing the debug window
… effectively lets just save to the users computer so that they can view it themselves
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated
LGTM |
changed based to unstable as we don't want this in the release with onboarding
Note: maybe we should just not display the logs and just copy them to desktop