-
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-alpha
and updated package metadata. - Added context-variable support and color formatting to the logger.
- Introduced
get_diff_files
ingit_utils
and changedcreate_and_analyze
signature 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_files
method. 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_name
tocreate_and_analyze
is 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 = False
can 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._fmt
is brittle and may lead to unexpected side effects. Consider creating a newFormatter
instance 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-alpha
inpyproject.toml
Related
v0.2.0-alpha
Checklist
This PR will be auto-merged after creation.