-
Notifications
You must be signed in to change notification settings - Fork 175
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
refactor: added type hints for Python SDK #665
refactor: added type hints for Python SDK #665
Conversation
@lohanidamodar, I think we should merge this and iterate upon it to add even more. |
Thanks @stnguyen90. |
@lohanidamodar I think we should do this for all templates at once, the service methods would benefit most from this |
@Ananya2001-an would you like to continue further to add type hints for the remaining parts? If not I'll merge your PR into a separate branch so that our team can work on top of it. Let me know ho you would like to proceed. |
Hey @lohanidamodar I would love to continue working on it....later on you can merge it to a separate branch when I have done the final changes according to me :) |
Hey, I think you can merge this for now since I am not sure where to add type hints further :) |
@Ananya2001-an not a problem. Thank you for your contribution |
What does this PR do?
Added type hints for better readability and maintainability. It's added only for a few methods at the moment but can improve on that later on... :)
Related PRs and Issues
With reference to issue: appwrite/sdk-for-python#43
Have you read the Contributing Guidelines on issues?
Yes