← Back to issue list

Pass the charmhub config to the Store API layer.

View original Github issue

Metadata

Project
charmcraft
Number
#227
Type
pull request
State
merged
Author
facundobatista
Labels
Created
Updated
Closed

Current evaluation

Merged. Passes charmhub configuration to the Store API layer through primarily mechanical changes with minimal functional impact. Relocates a test file. Approved by two reviewers and passed CI checks.

Suggested action:

No scores available.

Issue body

Almost all are mechanic changes, the functional impact is super low (the config still doesn't reach the store client, which is the one that changes). Also moved a test file (which I forgot to move when moved the real `utils.py` file).

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged. Passes charmhub configuration to the Store API layer through primarily mechanical changes with minimal functional impact. Relocates a test file. Approved by two reviewers and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged changes pass charmhub configuration to the Store API layer via mechanical updates. Functional impact remains low as the config does not yet reach the store client. Includes a relocated test file.
qwen3.6-35b-a3b-mtp-q6 Merged changes passing charmhub configuration to the Store API layer. Modifications are primarily mechanical with minimal functional impact, as the config does not yet reach the store client. Also relocates a previously misplaced test file.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#82 Work in preparation for the list-revisions branch. charmcraft merged Merged after approval by two reviewers. The changes isolate a helper function for extracting charm names from metadata, update related tests, and implement the new Store API base URL domain. Two files were modified with 82 additions and 45 deletions.
71%
#2659 docs: remove charmhub config charmcraft merged Merged into main. Documentation references to charmhub were removed while keeping the underlying code intact for backward compatibility. Closes issue 1787 after reviewer approval and CI validation.
71%