Skip to content

Fix/python tests#1

Merged
justinmadison merged 2 commits intomainfrom
fix/python-tests
Nov 7, 2025
Merged

Fix/python tests#1
justinmadison merged 2 commits intomainfrom
fix/python-tests

Conversation

@justinmadison
Copy link
Member

@justinmadison justinmadison commented Nov 7, 2025 •
edited
Loading

Description

Brief description of what this PR does.

Closes #(issue number)

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Performance improvement
  • Code refactoring
  • Tests
  • Build/CI changes

Changes Made

  • List key changes
  • One per line
  • Be specific

Testing

  • Tests pass locally (pytest tests/)
  • Added new tests for new features
  • Tested manually (describe below)
  • No regressions in existing functionality

Manual Testing:
Describe how you tested this change manually.

Performance Impact

  • No performance impact
  • Improves performance
  • May impact performance (explain below)

Documentation

  • Updated relevant documentation
  • Added code comments for complex logic
  • Updated CHANGELOG (if applicable)
  • Added/updated docstrings

Checklist

  • My code follows the project's style guidelines
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Screenshots (if applicable)

Add screenshots to help reviewers understand your changes.

Additional Notes

Any additional information reviewers should know.

justinmadison and others added 2 commits November 6, 2025 17:42
@justinmadison @claude
Fix Python test failures in GitHub CI
b75f8b7 
- Add pyproject.toml for proper package installation with setuptools
- Configure package structure with python/ as source directory
- Add tests/__init__.py to make tests directory importable
- Update GitHub Actions workflow to install package with `pip install -e .[dev]`
- Simplify test execution to run from project root
- Fix coverage report path from ./tests/coverage.xml to ./coverage.xml
- Update linting tools installation and execution paths
- Split dependencies into core, optional (llm, vector, full), and dev groups
- Configure pytest, black, ruff, and mypy in pyproject.toml

All 16 tests now pass locally with 69% code coverage.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@justinmadison
fix git directory
670845c
AndrewBMadison
AndrewBMadison approved these changes Nov 7, 2025
View reviewed changes
Copy link
Collaborator

@AndrewBMadison AndrewBMadison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@justinmadison justinmadison merged commit 82976c5 into main Nov 7, 2025
6 of 7 checks passed
This was referenced Feb 18, 2026
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndrewBMadison AndrewBMadison AndrewBMadison approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@justinmadison @AndrewBMadison