← Back to issue list

build(deps): remove protobuf dep

View original Github issue

Metadata

Project
craft-store
Number
#156
Type
pull request
State
merged
Author
lengau
Labels
Created
Updated
Closed

Current evaluation

Merged after removing the protobuf dependency, resolving issue #132. Approved by two reviewers, passed CI, and modified one file with a net reduction of three lines.

Suggested action:

No scores available.

Issue body

fixes #132 - [x] Have you followed the guidelines for contributing? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] Have you successfully run `tox`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged after removing the protobuf dependency, resolving issue #132. Approved by two reviewers, passed CI, and modified one file with a net reduction of three lines.
qwen3.6-35b-a3b-mtp-q6 Merged changes removing the protobuf dependency, resolving issue #132. Maintainer confirmed tox tests passed before integration.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2614 build(deps): downgrade protobuf for s390x segfault charmcraft merged Merged to downgrade protobuf and fix an s390x segfault. Approved by two reviewers and passed core CI checks. Changes affect two files with minor dependency version adjustments.
78%
#1675 build(deps): update all dependencies charmcraft merged Merged after two reviewers approved and CI passed. The dependency update modified three files with a net change of zero lines.
76%
#3923 build(deps): bump protobuf from 3.20.1 to 3.20.2 snapcraft merged Merged an automated Dependabot pull request bumping protobuf from 3.20.1 to 3.20.2. The update resolves a C++ and Python security advisory and reduces MessageSet parsing memory usage. Approved and merged successfully.
75%
#472 build(deps): update dependencies rockcraft merged Merged dependency updates across four files, modifying 57 additions and 58 deletions. Approved by two reviewers with passing CI checks. The maintainer-authored changes were integrated into the codebase without further comments.
74%
#1298 build(deps): update dependencies rockcraft merged Approved by two reviewers, the dependency update pull request was merged. Despite two initially failing CI checks, the changes across two files were successfully integrated.
74%
#1493 build(deps): update dependencies charmcraft merged Merged pull request updating project dependencies. Approved by two reviewers, the maintainer addressed comments on unused packages and clarified that CI failures resulted from an external issue.
74%
#1946 build(deps): update dependency protobuf to v5.28.2 (main) charmcraft merged Merged an automated dependency update upgrading protobuf from v5.27.3 to v5.28.2. Approved by two reviewers, passed CI checks, and merged into main with minimal changes.
73%
#4504 build(deps): update dependencies snapcraft merged Merged after approval by two reviewers. The dependency update modified three files, passed all CI and Codecov checks, and was successfully integrated into the main branch.
73%
#876 Bump protobuf from 3.20.1 to 3.20.2 charmcraft merged Merged Dependabot update upgrading protobuf from 3.20.1 to 3.20.2 to address a security advisory and reduce memory consumption. Approved by reviewer, passed CI, and successfully merged.
73%
#933 build(deps): update all dependencies rockcraft merged All dependencies were updated and merged after two reviewers approved and all CI checks passed. The change modified nine files, adding 494 lines and removing 446.
72%