-
Notifications
You must be signed in to change notification settings - Fork 0
feat: UV venv support. #17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.