← Back to issue list

build(deps): drop pyyaml dependency

View original Github issue

Metadata

Project
craft-cli
Number
#332
Type
pull request
State
merged
Author
tigarmo
Labels
Created
Updated
Closed

Current evaluation

Merged to drop the unused pyyaml dependency ahead of a major release. Approved by two reviewers, passed CI checks, and modified a single file.

Suggested action:

No scores available.

Issue body

We don't actually use it for anything, and since we're prepping for a major release we might as well drop it. - [ ] Have you followed the guidelines for contributing? - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [ ] Have you successfully run `tox`?

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged to drop the unused pyyaml dependency ahead of a major release. Approved by two reviewers, passed CI checks, and modified a single file.
qwen3.6-35b-a3b-mtp-q6 Merged removal of the unused pyyaml dependency ahead of a major release. The change streamlines the build configuration and reduces external package requirements without impacting functionality.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#811 build(deps): drop unnecessary pydantic-yaml craft-parts merged Merged to remove the pydantic-yaml dependency, as Pydantic 2 uses model_dump() with PyYAML directly. Approved by two reviewers and passed CI checks.
82%
#3490 build(deps): bump pyyaml from 5.3 to 5.4 snapcraft merged Merged Dependabot update bumping pyyaml from 5.3 to 5.4. Required test updates and a rebase were completed before approval and successful CI merge.
79%
#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%
#524 build(deps): bump pytest starbase merged Merged a dependency update bumping pytest to resolve an OSV. Approved by two reviewers with all CI checks passing. The change updates a single file.
72%
#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.
72%
#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.
71%
#156 build(deps): remove protobuf dep craft-store merged 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.
71%
#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.
70%
#321 build(deps): bump pytest imagecraft merged Merged a dependency update bumping pytest to resolve an OSV vulnerability. Approved by two reviewers with all CI checks passing. The change modified a single file with minimal adjustments.
70%
#3713 requirements: unpin pyyaml and update dependencies snapcraft merged Merged following reviewer approval and passing CI. Unpinned mccabe and PyYAML to enable libyaml bindings for Python 3.10, updating four dependency files.
70%