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

feat: support async execution #103

Merged
merged 4 commits into from Feb 16, 2022
Merged

feat: support async execution #103

merged 4 commits into from Feb 16, 2022

Conversation

lebovic
Copy link
Member

@lebovic lebovic commented Feb 16, 2022

Adds:

  • Support for async execution

See https://gist.github.com/lebovic/72fbb857119f1667c7959a4d7e28cd50 (or the integration test) for a hacky example on how to run Toolchest with async execution.

@lebovic lebovic requested a review from bcai2 February 16, 2022 00:05
tests/test_async.py Show resolved Hide resolved
toolchest_client/tools/tool.py Outdated Show resolved Hide resolved
toolchest_client/tools/api.py Show resolved Hide resolved
Copy link
Contributor

@bcai2 bcai2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! a couple typos, and I am mildly concerned about Query being circularly imported, but it seems justifiable to me over having a separate module just for checking the status

toolchest_client/api/download.py Outdated Show resolved Hide resolved
toolchest_client/api/status.py Outdated Show resolved Hide resolved
@bcai2
Copy link
Contributor

bcai2 commented Feb 16, 2022

Also let's hold off on merging/running integration tests until the bugfix is merged

@lebovic lebovic merged commit 531cf1c into staging Feb 16, 2022
@lebovic lebovic deleted the feat/async branch February 16, 2022 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants