search

aai-institute / jobq

https://aai-institute.github.io/jobq/latest
Apache License 2.0
2 stars 1 forks source link

Import docker only in the `ResourceOptions.to_docker()` method #127

Closed nicholasjng closed 1 month ago

nicholasjng commented 1 month ago

This removes the need for docker as an intra-container dependency.

Not sure if putting the type hint into quotes is sufficient - local mypy showed me failures, but I also messed up my venv I think.

AdrianoKF commented 1 month ago

I think you'll need to fix line 82 as well (where it's being used for typing).