Skip to content

improve: harden gitignore and remove QA temp artifacts#47

Open
Coding-Dev-Tools wants to merge 1 commit into
masterfrom
improve/envault-1782904256
Open

improve: harden gitignore and remove QA temp artifacts#47
Coding-Dev-Tools wants to merge 1 commit into
masterfrom
improve/envault-1782904256

Conversation

@Coding-Dev-Tools

Copy link
Copy Markdown
Owner

Automated improvement by dev-engineer

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

🤖 Automated Code Review

⚠️ Ruff Lint — 6 issue(s)

::error title=ruff (I001),file=/home/runner/work/envault/envault/tests/test_cli_edge_cases.py,line=12,endLine=18::tests/test_cli_edge_cases.py:12:1: I001 Import block is un-sorted or un-formatted%0A  help: Organize imports
::error title=ruff (I001),file=/home/runner/work/envault/envault/tests/test_encrypt_formats.py,line=7,endLine=9::tests/test_encrypt_formats.py:7:1: I001 Import block is un-sorted or un-formatted%0A  help: Organize imports
::error title=ruff (I001),file=/home/runner/work/envault/envault/tests/test_encrypt_secret_formats.py,line=7,endLine=10::tests/test_encrypt_secret_formats.py:7:1: I001 Import block is un-sorted or un-formatted%0A  help: Organize imports
::error title=ruff (I001),file=/home/runner/work/envault/envault/tests/test_history.py,line=3,endLine=7::tests/test_history.py:3:1: I001 Import block is un-sorted or un-formatted%0A  help: Organize imports
::error title=ruff (F403),file=/home/runner/work/envault/envault/tests/test_history.py,line=6,col=1,endLine=6,endColumn=30::tests/test_history.py:6:1: F403 `from envault.history import *` used; unable to detect undefined names
::error title=ruff (I001),file=/home/runner/work/envault/envault/tests/test_security_audit.py,line=3,endLine=12::tests/test_security_audit.py:3:1: I001 Import block is un-sorted or un-formatted%0A  help: Organize imports

⚠️ Ruff Format — Formatting needed

Would reformat: src/envault/audit.py
Would reformat: src/envault/auth.py
Would reformat: src/envault/backup.py
Would reformat: src/envault/cli.py
Would reformat: src/envault/config.py
Would reformat: src/envault/diff.py
Would reformat: src/envault/encrypt.py
Would reformat: src/envault/history.py
Would reformat: src/envault/rotate.py
Would reformat: src/envault/security_audit.py
Would reformat: src/envault/serve.py
Would reformat: src/envault/stores/__init__.py
Would reformat: src/envault/sync.py
Would reformat: tests/test_cli.py
Would reformat: tests/test_cli_edge_cases.py
Would reformat: tests/test_coverage_gaps.py
Would reformat: tests/test_encrypt_formats.py
Would reformat: tests/test_encrypt_secret_formats.py
Would reformat: tests/test_envault.py
Would reformat: tests/test_lint_regression.py
Would reformat: tests/test_security_audit.py
Would reformat: tests/test_serve.py
Would reformat: tests/test_stores_integration.py
Would reformat: tests/test_sync_coverage.py
24 files would be reformatted, 3 files already formatted

🔴 Secret Detection — 1 potential secret(s)

  src/envault/serve.py:36 — Secret Keyword

✅ Large Files — Within limits

📊 Diff Stats — 4 file(s) changed

 .gitignore        |  6 +++++
 _qa_repro_test.py | 52 -------------------------------------
 _qa_test_scan.py  | 77 -------------------------------------------------------
 test-file.txt     |  1 -
 4 files changed, 6 insertions(+), 130 deletions(-)

Verdict: 🔴 Changes Requested — Potential secrets detected. Do not merge.

Automated by Coding-Dev-Tools/.github reusable workflow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant