ansible / ansible-sdk

The Ansible SDK
Apache License 2.0
24 stars 14 forks source link

[wip] Add mypy to linters job #80

Closed Shrews closed 1 year ago

Akasurde commented 1 year ago

@Shrews Thanks for putting this together. I added some type-related changes in https://github.com/ansible/ansible-sdk/pull/78/files#diff-67d11139be7ac36ebd1291ecf5d4e801ecade35e4564d8921013d3d39f43a9f4R38

Shrews commented 1 year ago

@Shrews Thanks for putting this together. I added some type-related changes in https://github.com/ansible/ansible-sdk/pull/78/files#diff-67d11139be7ac36ebd1291ecf5d4e801ecade35e4564d8921013d3d39f43a9f4R38

I put this up to get an idea of how much work is needed to get this job added. This will never be mergable unless we work on fixing all of the current issues. That task is a bit large for me to attack in this PR. It's only going to get worse as more code is added.