← Back to issue list

refactor(utils): move charmlibs utilities

View original Github issue

Metadata

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

Current evaluation

Merged a refactor moving charmlibs utilities across eight files, adding 61 lines and removing 42. Approved by two reviewers with zero unresolved comments.

Suggested action:

No scores available.

Issue body

No description provided.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a refactor moving charmlibs utilities across eight files, adding 61 lines and removing 42. Approved by two reviewers with zero unresolved comments.
qwen3.6-35b-a3b-mtp-q6 Merged a refactoring update that relocates charmlibs utility functions. The maintainer integrated the change directly without requiring additional discussion or modifications.
qwen3.6-35b-a3b-mtp-q6 Merged a refactoring change that relocated charmlibs utility functions to a new directory structure. The update was integrated by maintainer lengau without review comments or revisions.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1357 refactor: reorganise constants charmcraft merged Merged a refactor reorganizing constants for Charmcraft environment variables, project filenames, and output files/directories across 47 files. Changes include 471 additions and 443 deletions. Approved by four reviewers with no unresolved comments.
74%
#993 refactor: Move charm library helper functions to their own modules (CRAFT-1557) charmcraft merged Merged refactoring extracting charm library helpers into dedicated modules, resolving CRAFT-1557. Approved by one reviewer with no unresolved comments. Modifies seven files, adding 658 lines and removing 587.
74%
#1301 refactor(utils): break utils sections out into logical subpackages charmcraft merged 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).
72%
#1085 charmlibs: skip non top-level name assignments charmcraft merged Merged a charmlibs fix for incorrect non-top-level assignment parsing. Filters ast.Names during ast.Assign evaluation to ignore chained attribute assignments. Approved by one reviewer and merged with a +15/-1 diff across two files.
70%