← Back to issue list

perf: cache project file path

View original Github issue

Metadata

Project
craft-application
Number
#684
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged a performance optimization caching the project file path. Approved by two reviewers, passed CI, and changed one line. The author kept the perf commit type as it improves efficiency rather than fixing a bug.

Suggested action:

No scores available.

Issue body

- [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `make lint && make test`? - [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)? --- Needs #683.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged a performance optimization caching the project file path. Approved by two reviewers, passed CI, and changed one line. The author kept the perf commit type as it improves efficiency rather than fixing a bug.
qwen3.6-35b-a3b-mtp-q6 Merged a performance optimization that caches the project file path. The change depends on PR #683 and was classified as a performance improvement rather than a bug fix.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#5748 fix: get the project from the project service every time snapcraft merged Merged a fix ensuring the project is retrieved from the project service on every call. Approved by two reviewers, the change modifies one file (+9/-1 lines) and resolves the retrieval logic despite one failing CI test.
71%