← Back to issue list

refactor(utils): break utils sections out into logical subpackages

View original Github issue

Metadata

Project
charmcraft
Number
#1301
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged a refactor splitting utils.py into logical subpackages while maintaining a backwards-compatible API. Approved by two reviewers, passed CI, and modified 18 files (+1588/-1333 lines).

Suggested action:

No scores available.

Issue body

This affects `utils.py`, which is now broken out into logical sections, with an `__init__.py` creating a consistent, backwards-compatible package API. Anything outside of `utils` and its corresponding tests does not need to change, as these changes do not affect the API seen by outside modules.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a refactor splitting utils.py into logical subpackages while maintaining a backwards-compatible API. Approved by two reviewers, passed CI, and modified 18 files (+1588/-1333 lines).
qwen3.6-35b-a3b-mtp-q6 Merged a refactoring that splits utils.py into logical subpackages while preserving a backwards-compatible API through __init__.py. External code requires no changes.
qwen3.6-35b-a3b-mtp-q6 Merged a refactoring splitting utils.py into logical subpackages while preserving a backwards-compatible API via __init__.py. No external changes required.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#174 tests(utils): improve coverage of utils tests fetch-service merged Merged following approval by two reviewers and passing CI checks. The update improved coverage of utils tests, with a diff showing zero lines changed, suggesting a metadata or squash merge.
72%
#1303 refactor(utils): move charmlibs utilities charmcraft merged Merged a refactor moving charmlibs utilities across eight files, adding 61 lines and removing 42. Approved by two reviewers with zero unresolved comments.
72%
#496 refactor: Reduce cyclomatic complexity fetch-service merged Merged refactoring changes that reduced cyclomatic complexity by extracting logic into dedicated functions. Approved by two reviewers, passed CI checks, and modified four files (+248/-201 lines).
71%