Skip to content
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

mypy typing expansion #206

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from
Draft

mypy typing expansion #206

wants to merge 7 commits into from

Conversation

DrPyser
Copy link
Contributor

@DrPyser DrPyser commented Aug 9, 2024

  • Added conventional mypy config
  • Added dependencies to mypy precommit env
  • Updated mypy linter pre-commit config
  • Added type stubs to test-requirements.txt
  • Updated mypy config
  • Added type hints for plugins.config, plugins.config_service and related modules

using --python-executable option to use tox environment to use type hints from dependencies directly
since pre-commit additional_dependencies are not used
expanded type checking to new module
warn about missing types for imports
@DrPyser DrPyser added mergeit and removed mergeit labels Aug 9, 2024
@DrPyser DrPyser marked this pull request as draft August 9, 2024 21:51
Copy link
Contributor

Build failed.
https://zuul.wazo.community/zuul/t/local/buildset/b11b61a57da849bba0ffd837798c9d13

tox-linters FAILURE in 8m 25s
✔️ wazo-tox-py39 SUCCESS in 8m 10s
✔️ debian-packaging-bullseye SUCCESS in 2m 23s
dird-tox-integration FAILURE in 34m 40s

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.

1 participant