feat: asyncio microgen client#118
Conversation
tritone
left a comment
There was a problem hiding this comment.
Generally looks good, a couple questions. Do you have system tests running yet which exercise the async microgen code path?
BenWhitehead
left a comment
There was a problem hiding this comment.
Good to merge after addressing @tritone's questions.
|
@tritone Unfortunately system tests do not exist yet - I was waiting to complete integration of the client before writing system tests, as I feel as though they are dependent on the entire client being converted to async, and it would be preferable to wait until that point. As this code is being committed to |
Discussed offline, I'm okay with merging this code into the branch before system tests are added. |
Resolves #73