top: Enable ruff linter check for F821 undefined-name.

Very helpful for catching typos or missing imports when writing code!

Description can be found at
https://beta.ruff.rs/docs/rules/undefined-name/

Parent commits contain various small fixes and inline ignores for this
check.  The only blanket exception is manifest files, which are numerous
and evaluated with some global names pre-defined - these can be globally
ignored.

Signed-off-by: Angus Gratton <angus@redyak.com.au>
This commit is contained in:
Angus Gratton 2023-08-09 18:24:29 +10:00 committed by Damien George
parent 1a5c9b9da4
commit 40fcbe1246

View File

@ -35,7 +35,6 @@ ignore = [
"F401",
"F403",
"F405",
"F821",
"PLC1901",
]
line-length = 337
@ -46,3 +45,7 @@ max-complexity = 40
[tool.ruff.per-file-ignores]
"ports/cc3200/tools/uniflash.py" = ["E711"]
# manifest.py files are evaluated with some global names pre-defined
"**/manifest.py" = ["F821"]
"ports/**/boards/manifest*.py" = ["F821"]