diff options
author | Daniel Latypov <dlatypov@google.com> | 2023-03-17 01:06:38 +0300 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2023-03-17 21:28:30 +0300 |
commit | 1da2e6220e1115930694c649605534baf6fa3dea (patch) | |
tree | 6716ece2ebaf7d125f7dca7e50ce34b144f70bd5 /include/kunit | |
parent | 126901ba3499880c9ed033633817cf7493120fda (diff) | |
download | linux-1da2e6220e1115930694c649605534baf6fa3dea.tar.xz |
kunit: tool: fix pre-existing `mypy --strict` errors and update run_checks.py
Basically, get this command to be happy and make run_checks.py happy
$ mypy --strict --exclude '_test.py$' --exclude qemu_configs/ ./tools/testing/kunit/
Primarily the changes are
* add `-> None` return type annotations
* add all the missing argument type annotations
Previously, we had false positives from mypy in `main()`, see commit
09641f7c7d8f ("kunit: tool: surface and address more typing issues").
But after commit 2dc9d6ca52a4 ("kunit: kunit.py extract handlers")
refactored things, the variable name reuse mypy hated is gone.
Note: mypy complains we don't annotate the types the unused args in our
signal handler. That's silly.
But to make it happy, I've copy-pasted an appropriate annotation from
https://github.com/python/typing/discussions/1042#discussioncomment-2013595.
Reported-by: Johannes Berg <johannes.berg@intel.com>
Link: https://lore.kernel.org/linux-kselftest/9a172b50457f4074af41fe1dc8e55dcaf4795d7e.camel@sipsolutions.net/
Signed-off-by: Daniel Latypov <dlatypov@google.com>
Reviewed-by: David Gow <davidgow@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'include/kunit')
0 files changed, 0 insertions, 0 deletions