-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Removing spinnies library from venom #2820
Comments
@9cb14c1ec0 good idea! Let me test it out |
?? |
Would also love to get some clean logging that's functional, even if its not as pretty. @allanalcantara that doesn't seem to make much of a difference for me.
|
Appologies it's a little verbose as I'm doing other things in the middle (I've tried to remove repeated code to shorten, like all the cases in the switch where I handle websocket connections differently)
|
weird.. I just created it, and it showed me off...
|
I'm sorry I don't understand? I turned debug From your initial screenshot, can you show me the lines above the qr data in your terminal (like shown in mine)? I want to see what prints in between |
Currently venom uses a third party library to add some eye-candy to the terminal output via progress spinners. This interferes automated log shipping and parsing. If I open a pull request to remove spinnies while retaining all other logged info, would it get merged?
The text was updated successfully, but these errors were encountered: