fix: resolve ruff F821/F401 lint errors in cli.py, serve.py, diff.py #8
auto-code-review.yml
on: push
code-review
/
Automated code review
12s
Annotations
21 errors and 1 warning
|
code-review / Automated code review
Process completed with exit code 1.
|
|
ruff (F811):
tests/test_cli.py#L243
tests/test_cli.py:243:5: F811 Redefinition of unused `test_diff_files_json_identical` from line 183: `test_diff_files_json_identical` redefined here
tests/test_cli.py:183:5: previous definition of `test_diff_files_json_identical` here
help: Remove definition: `test_diff_files_json_identical`
|
|
ruff (F811):
tests/test_cli.py#L225
tests/test_cli.py:225:5: F811 Redefinition of unused `test_diff_files_json_output` from line 164: `test_diff_files_json_output` redefined here
tests/test_cli.py:164:5: previous definition of `test_diff_files_json_output` here
help: Remove definition: `test_diff_files_json_output`
|
|
ruff (I001):
src/envault/serve.py#L17
src/envault/serve.py:17:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
ruff (SIM102):
src/envault/security_audit.py#L450
src/envault/security_audit.py:450:9: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (SIM102):
src/envault/security_audit.py#L297
src/envault/security_audit.py:297:5: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (SIM102):
src/envault/security_audit.py#L167
src/envault/security_audit.py:167:13: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (F401):
src/envault/diff.py#L7
src/envault/diff.py:7:8: F401 `os` imported but unused
help: Remove unused import: `os`
|
|
ruff (E402):
_qa_test_scan.py#L35
_qa_test_scan.py:35:1: E402 Module level import not at top of file
|
|
ruff (I001):
_qa_test_scan.py#L2
_qa_test_scan.py:2:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
ruff (I001):
_qa_repro_test.py#L2
_qa_repro_test.py:2:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
ruff (F811):
tests/test_cli.py#L243
tests/test_cli.py:243:5: F811 Redefinition of unused `test_diff_files_json_identical` from line 183: `test_diff_files_json_identical` redefined here
tests/test_cli.py:183:5: previous definition of `test_diff_files_json_identical` here
help: Remove definition: `test_diff_files_json_identical`
|
|
ruff (F811):
tests/test_cli.py#L225
tests/test_cli.py:225:5: F811 Redefinition of unused `test_diff_files_json_output` from line 164: `test_diff_files_json_output` redefined here
tests/test_cli.py:164:5: previous definition of `test_diff_files_json_output` here
help: Remove definition: `test_diff_files_json_output`
|
|
ruff (I001):
src/envault/serve.py#L17
src/envault/serve.py:17:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
ruff (SIM102):
src/envault/security_audit.py#L450
src/envault/security_audit.py:450:9: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (SIM102):
src/envault/security_audit.py#L297
src/envault/security_audit.py:297:5: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (SIM102):
src/envault/security_audit.py#L167
src/envault/security_audit.py:167:13: SIM102 Use a single `if` statement instead of nested `if` statements
help: Combine `if` statements using `and`
|
|
ruff (F401):
src/envault/diff.py#L7
src/envault/diff.py:7:8: F401 `os` imported but unused
help: Remove unused import: `os`
|
|
ruff (E402):
_qa_test_scan.py#L35
_qa_test_scan.py:35:1: E402 Module level import not at top of file
|
|
ruff (I001):
_qa_test_scan.py#L2
_qa_test_scan.py:2:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
ruff (I001):
_qa_repro_test.py#L2
_qa_repro_test.py:2:1: I001 Import block is un-sorted or un-formatted
help: Organize imports
|
|
code-review / Automated code review
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-python@v5. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|