ci: generate coverage.xml for Codecov and align Makefile with uv-run workflow #13
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Update Makefile coverage target to generate XML report for codecov - Add --cov-report=xml flag for CodeCov integration - Add --cov-report=term-missing for better local development feedback - Simplify test command to use direct pytest execution 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Removes the -i (interactive) flag from shell command execution which can cause issues in automated environments like CI/CD pipelines.
…rs from pyproject
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
…xecutor - Add ProcessManager tests for is_running attribute, cleanup_all behavior, timeout handling, and signal handling - Add ShellExecutor tests for shell fallback, pipeline validation, command parsing errors, and IO redirection - Improve test coverage for error handling paths and edge cases in both components 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Expand edge case testing in test_shell_executor_edge_cases.py - Improve test coverage in test_process_manager_additional.py - Add comprehensive test coverage in test_shell_executor_more_coverage.py - Minor improvements to shell_executor.py for better testability 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
…ports in tests to satisfy CI lint
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR fixes the failing GitHub Actions run by ensuring Codecov finds a coverage report.
Changes:
--cov-report=xmland add--cov-report=term-missinguv runfrom test/coverage targets to avoid doubleuv run; CI continues to calluv run make ...Local verification:
Expected result: