Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds support for UV virtual environments by updating the code standard module detection logic to check both uv pip list and the standard pip list commands. The changes ensure that the tool can detect installed packages (ruff/flake8) in UV-managed environments.
Key changes:
- Modified
_get_available_code_standard_module()to tryuv pip listbefore falling back to standardpip list - Enhanced test coverage for the updated detection logic with multiple scenarios
- Refactored tests to use the new
_run_code_standard_tests()helper function
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| mfd_code_quality/code_standard/checks.py | Added UV pip list command as primary detection method with fallback to standard pip |
| tests/unit/test_mfd_code_quality/test_code_standard/test_checks.py | Updated tests to cover UV detection scenarios and refactored to test the new helper function |
| tests/unit/test_mfd_code_quality/test_mfd_code_quality.py | Added mocker fixture and mocking to prevent argparse from interfering with test execution |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
We don't publish DEVs .whl. |
aa0f816 to
ac14215
Compare
Signed-off-by: Mateusz Chrominski <mateusz.chrominski@intel.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.