-
Notifications
You must be signed in to change notification settings - Fork 296
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
Bug: localnpipe translation not working in v3 #600
Comments
Found a fix, just It took me longer than it should've to find the fix because all I could see was Where I found the fix #108 (comment) |
Hang on... in waiting_utils.py it does log, but setup_logger enforces logging config, as a package this should not be happening? Logging config should be up to application not the package |
I'm slightly confused about why the TC_HOST fix was necessary despite the fix in #427 |
@alexanderankin version? I had 3.7.1 |
The v3 line is from 2022/jan 2023, the first release of v4 was this year with all new maintainers of the project |
@fraser-langton if you run into difficulties with an up-to-date version, please open a new issue. |
@alexanderankin what about changing the usage of setup_logger which made this issue harder to debug, worthwhile fixing? Make a different issue? |
would you consider that breaking? or any ideas about how to soften the breaking of that change (config flag :((( ?) - I put it on my v5 list #631 |
Describe the bug
Cannot do the most basic use case
To Reproduce
I copied the example from the docs word for word
Runtime environment
Windows 11 + Python 3.8.10
systeminfo
docker info
freeze
The text was updated successfully, but these errors were encountered: