Skip to content

fix: resolve ruff F821/F401 lint errors in cli.py, serve.py, diff.py #8

fix: resolve ruff F821/F401 lint errors in cli.py, serve.py, diff.py

fix: resolve ruff F821/F401 lint errors in cli.py, serve.py, diff.py #8

Triggered via push June 29, 2026 02:28
Status Failure
Total duration 16s
Artifacts
code-review  /  Automated code review
12s
code-review / Automated code review
Fit to window
Zoom out
Zoom in

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/