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

Add asynchronous support #94

Closed
wants to merge 5 commits into from
Closed

Add asynchronous support #94

wants to merge 5 commits into from

Conversation

py-mine-bot
Copy link
Collaborator

5IGI0 Authored by 5IGI0
Oct 24, 2020
Merged Oct 29, 2020


i'm back ^^

asynchronous DNS resolving and asynchronous query isn't implemented for now, otherwise everything is done!

@py-mine-bot
Copy link
Collaborator Author

5IGI0 Authored by 5IGI0
Oct 29, 2020


i wanted to add a test to check that the reader functions are asynchronous, but the dir() function does not return the attributes of the class '-'

image
image

@py-mine-bot
Copy link
Collaborator Author

kevinkjt2000 Authored by kevinkjt2000
Oct 29, 2020


That's a test that I wouldn't have even thought of, very nice! Did you peek at dir(TCPAsyncSocketConnection)?

@py-mine-bot
Copy link
Collaborator Author

5IGI0 Authored by 5IGI0
Oct 29, 2020


That's a test that I wouldn't have even thought of, very nice! Did you peek at dir(TCPAsyncSocketConnection)?7

already tried, it doesin't work ;'c
but i can try to create an list of attributes outside the function

@py-mine-bot
Copy link
Collaborator Author

5IGI0 Authored by 5IGI0
Oct 29, 2020


i'm just the dumbest guy on earth, i named the test class with the same name that the connection class

@py-mine-bot py-mine-bot added the Github Import This was auto-imported from upstream repository label Feb 17, 2022
@py-mine-bot py-mine-bot deleted the pr94head branch February 17, 2022 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Github Import This was auto-imported from upstream repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants