Skip to content

Conversation

@ckrough
Copy link
Owner

@ckrough ckrough commented Dec 20, 2025

Summary

  • Add GitHub Actions CI pipeline with parallel jobs for test, lint, type-check, and security scanning
  • Add pytest-cov dependency for coverage reporting in CI
  • Fix documentation accuracy issues across multiple files
  • Fix lint/format issues in test files to pass CI checks

Changes

CI Pipeline (.github/workflows/ci.yml)

  • Test: pytest with coverage reporting
  • Lint: ruff format and check
  • Type Check: mypy strict mode
  • Security: bandit scan

Documentation Fixes

  • Update repository URL in README
  • Fix CLI description in CONTRIBUTING.md (add evaluate command)
  • Fix ground truth example domain in CONTRIBUTING.md
  • Update ADR-001 with maintainable function reference instead of line numbers
  • Update ADR-002 with current Anthropic model naming
  • Add status badges to README (CI, Python version, license)

Code Quality

  • Add pytest-cov>=4.0 to dev dependencies
  • Fix unused imports in test files
  • Fix line-too-long errors using string concatenation
  • Apply ruff formatting to test files

Test plan

  • All local checks pass: pytest, ruff, mypy, bandit
  • CI pipeline runs successfully on this PR
  • Coverage report is generated and uploaded as artifact

- Add CI workflow with parallel jobs: test, lint, type-check, security
- Add status badges to README (CI, Python version, license)
- Fix repository URL placeholder in README
- Fix CONTRIBUTING.md: add evaluate command, correct ground truth example
- Fix ADR-001: use function reference instead of stale line numbers
- Fix ADR-002: update to current Anthropic model naming
- Add pytest-cov>=4.0 to dev dependencies for coverage reporting
- Fix unused imports in test_actions_base.py and test_tag_manager.py
- Fix line-too-long errors in test files using string concatenation
- Apply ruff formatting to 6 test files
Add explicit ModuleType annotation for _xattr class variable so mypy
can determine the type on Linux CI where xattr module isn't available.
@ckrough ckrough merged commit 02d674e into main Dec 20, 2025
4 checks passed
@ckrough ckrough deleted the brave-wozniak branch December 20, 2025 20:40
@ckrough ckrough restored the brave-wozniak branch December 21, 2025 12:39
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.

2 participants