Skip to content

add type annotations to client/__init__.py #389

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

Merged

Conversation

junkmd
Copy link
Collaborator

@junkmd junkmd commented Nov 29, 2022

related to #327

There are functions in this module those in some cases return lazybind.Dispatch.
However,

  • lazybind module does not yet have type annotations.
  • lazybind.Dispatch is an object that implements __getattr__ and __setattr__, the type checker will not error even if specifying a non-existing member.

So for now, each function is annotated to return Any if the interface is not explicitly specified in the argument.

junkmd added a commit to junkmd/pywinauto that referenced this pull request Nov 29, 2022
@junkmd
Copy link
Collaborator Author

junkmd commented Nov 29, 2022

@vasily-v-ryabov

I requested your review because I think it would be healthier for the community to have you review the PRs I submit.


This PR is the last of adding type annotations to the statically defined modules mentioned in #327.

As next step, or at the same time, change of the codegenerator will be started, including #385.

junkmd added a commit to junkmd/pywinauto that referenced this pull request Nov 30, 2022
@vasily-v-ryabov vasily-v-ryabov merged commit ea26c1d into enthought:master Dec 3, 2022
@junkmd junkmd deleted the add_annotations_to_client_init branch December 4, 2022 02:08
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.

2 participants