-
Notifications
You must be signed in to change notification settings - Fork 0
Release v0.2.0-alpha #52
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
…tags before creating new ones
- Bump version to 0.2.0-alpha - Update package metadata Released via GitHub Actions
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
Automated release bump to version 0.2.0-alpha, which also includes infrastructure updates to logging, Git utilities, and the CodeQL runner.
- Bumped version to
0.2.0-alphaand updated package metadata. - Added context-variable support and color formatting to the logger.
- Introduced
get_diff_filesingit_utilsand changedcreate_and_analyzesignature in the CodeQL runner. - Removed existing test suite files (tests coverage dropped).
Reviewed Changes
Copilot reviewed 23 out of 23 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/codeql_wrapper/infrastructure/logger.py | Added current_log_color and current_format, enhanced ShortNameFormatter for color and context, adjusted get_logger and project context functions. |
| src/codeql_wrapper/infrastructure/git_utils.py | Added get_diff_files static method to list changed files between two refs. |
| src/codeql_wrapper/infrastructure/codeql_runner.py | Broke API of create_and_analyze by introducing project_name param and changing sarif_category. |
| tests/… | Bulk deletion of all test files. |
Comments suppressed due to low confidence (5)
src/codeql_wrapper/infrastructure/git_utils.py:70
- No tests cover the new
get_diff_filesmethod. Consider adding unit tests for scenarios like no changes, added/modified/deleted files, and handling of subprocess failures.
def get_diff_files(
src/codeql_wrapper/infrastructure/codeql_runner.py:165
- Adding
project_nametocreate_and_analyzeis an API-breaking change; existing callers will fail. Please update all call sites or provide a backward-compatible overload.
project_name: str,
src/codeql_wrapper/infrastructure/logger.py:93
- [nitpick] Removing explicit
logger.propagate = Falsecan cause log messages to be emitted twice. Consider disabling propagation on custom handlers to avoid duplicate output.
if format_string is None:
src/codeql_wrapper/infrastructure/logger.py:51
- [nitpick] Directly mutating the formatter’s internal
_style._fmtis brittle and may lead to unexpected side effects. Consider creating a newFormatterinstance when the format needs to change.
if current_fmt and current_fmt != self._style._fmt:
Release v0.2.0-alpha
This is an automated release pull request created by the release workflow.
Changes
0.2.0-alphainpyproject.tomlRelated
v0.2.0-alphaChecklist
This PR will be auto-merged after creation.