Skip to content

Type hints #7

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
merged 15 commits into from
Jun 3, 2025
Merged

Type hints #7

merged 15 commits into from
Jun 3, 2025

Conversation

RyanMcCleary
Copy link

This PR adds type hints to all methods, functions, and variables. Additionally, it adds an extra path for pyright to find NVDA source files, and adds documentation discussing this change in the README.

@seanbudd seanbudd merged commit ee541ee into nvaccess:main Jun 3, 2025
3 checks passed
@RyanMcCleary RyanMcCleary deleted the type-hints branch June 5, 2025 23:18
seanbudd added a commit that referenced this pull request Jun 24, 2025
* Type hints (#7)

This PR adds type hints to all methods, functions, and variables. Additionally, it adds an extra path for pyright to find NVDA source files, and adds documentation discussing this change in the README.

* Update translations

* Docstrings (#8)



This PR adds docstrings to classes, methods, and functions.

* Update translations

---------

Co-authored-by: RyanMcCleary <[email protected]>
Co-authored-by: github-actions <[email protected]>
seanbudd added a commit that referenced this pull request Jun 24, 2025
* Type hints (#7)

This PR adds type hints to all methods, functions, and variables. Additionally, it adds an extra path for pyright to find NVDA source files, and adds documentation discussing this change in the README.

* Update translations

* Docstrings (#8)



This PR adds docstrings to classes, methods, and functions.

* Update translations

---------

Co-authored-by: RyanMcCleary <[email protected]>
Co-authored-by: github-actions <[email protected]>
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