providers: add get_instance_name() for naming instances (CRAFT-310)
Metadata
Current evaluation
Merged addition of get_instance_name() for instance naming (CRAFT-310). Approved by two reviewers with zero unresolved comments. Introduces 54 lines across two files and was signed off by maintainer Chris Patterson.
Suggested action: —
No scores available.
Issue body
Signed-off-by: Chris Patterson <chris.patterson@canonical.com>
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged addition of get_instance_name() for instance naming (CRAFT-310). Approved by two reviewers with zero unresolved comments. Introduces 54 lines across two files and was signed off by maintainer Chris Patterson. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged without comments or revisions, adding get_instance_name() to providers for instance naming and resolving CRAFT-310. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds get_instance_name() to providers for instance naming, resolving CRAFT-310. Author cjp256 submitted the change, which was accepted and integrated without comments or revisions. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #885 (internal) Moved get_instance_name functionality to providers helper (CRAFT-1304). | charmcraft | merged | Merged refactoring moving get_instance_name to a providers helper, resolving CRAFT-1304. Approved by one reviewer with no unresolved comments, spanning eight files with a net reduction of 49 lines. |