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

Surpress any errors when using rdp and broken python version #139

Merged
merged 3 commits into from
Jan 22, 2024

Conversation

NeffIsBack
Copy link
Contributor

@NeffIsBack NeffIsBack commented Dec 7, 2023

This will surpress anything printed to stderr when using rdp and a broken python version because asyncio is broken in some versions, so fixes #73.
When a new python version is released containing the fix in asyncio we can remove the code block.

@NeffIsBack NeffIsBack added the bug-fix This Pull Request fixes a bug label Dec 7, 2023
@NeffIsBack NeffIsBack added reviewed code Label for when a static code review was done requires testing labels Dec 22, 2023
@Marshall-Hallenbeck
Copy link
Collaborator

Ran tests and they passed 👍

@Marshall-Hallenbeck Marshall-Hallenbeck merged commit f058c2d into main Jan 22, 2024
2 checks passed
@Marshall-Hallenbeck Marshall-Hallenbeck deleted the neff-asyncio branch January 22, 2024 20:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug-fix This Pull Request fixes a bug requires testing reviewed code Label for when a static code review was done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RDP not working as expected
3 participants