nvda
b47fdc52 - Ruff changes, expand pre-commit hooks to checkPot and unit tests (#16767)

Commit
1 year ago
Ruff changes, expand pre-commit hooks to checkPot and unit tests (#16767) Fix up for #16751 Summary of the issue: Ruff recently updated, so we might as well use the latest version. a warning is omitted for the section we added logger-objects to in ruff's config. it was unclear in docs how to avoid pre-commit hooks from triggering. more pre-commit hooks could be added ruff whitespace formating is done through ruff format currently we are only linting with ruff check. We should also run a ruff format on the whole NVDA. ruff was missing scons files, which should also be linted in a mass lint. Summary of other changes: Added pre commit hooks for unit tests and translation comment checks. Added generic pre-commit hooks for python static syntax checking, basic whitespace rules, and file write safety checks change the output of rununittests.bat to have simpler buffer output, only show failures. Ensure the verbose log is logged in AppVeyor. add ruff format to lint checks add scons files to lint checks update coderabbit prompt for change log entry items
Author
Parents
Loading