-
Notifications
You must be signed in to change notification settings - Fork 102
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
chore: pull types from wip PR for upcoming Model.wait_for_idle #1203
chore: pull types from wip PR for upcoming Model.wait_for_idle #1203
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, just one potential blocker -- behaviour change for Model.info
which I guess just means either correct the docstring or revert the behaviour change (and change the type annotation accordingly).
43d1935
to
99d8fc1
Compare
re self._info, pls reload |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good, just one docstring inaccuracy that could be fixed before merging
I'll need to either wait for canonical/pylxd#615 Or rip out the tox.ini and deps changes and chuck them aside... to a separate PR. |
/merge |
Improve type hints.
Sourced from #1104
These type hints will be used to upstream the new wait_for_idle()