← 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 PR #332 (commit 790e307) removed the unused PyYAML dependency from craft-cli's packaging metadata ahead of the planned major release.

Resolution: closed resolved

Reason: PR #332 was approved by 2 reviewers and merged as commit 790e307 "build(deps): drop pyyaml dependency (#332)", deleting the single PyYAML line from the project metadata.

Evaluated by: mr-cal-bot

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
stealth/space-bunny-alpha — closed resolved Merged PR #332 (commit 790e307) removed the unused PyYAML dependency from craft-cli's packaging metadata ahead of the planned major release.
stealth/space-bunny-alpha — closed resolved Merged canonical/craft-cli#332 to remove the unused PyYAML dependency ahead of a major release.
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.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-cli#137 (confidence 30%)

    Earlier dependency-metadata maintenance in the same packaging file (setup.cfg, later consolidated into pyproject.toml)

Related issues

Issue Project State Summary Similarity
#207 chore(deps): drop Pydantic dependency craft-cli merged PR #207 by tigarmo removed the unused Pydantic dependency from craft-cli's build config (+0/-3 lines across 3 files) and closed issue #201. Merged as commit ce2ee08 after approval from 2 reviewers; Codecov reported no coverage change.
73%
#811 build(deps): drop unnecessary pydantic-yaml craft-parts merged Merged #811 in commit 1debce16, removing pydantic-yaml because Pydantic 2 models are serialized with model_dump() and PyYAML directly.
73%
#5636 build(deps): use DEV craft-cli snapcraft closed PR to use a DEV channel craft-cli was closed unmerged as superseded by PR #5626, which was squash-merged as 932bd92e5 updating craft-cli to 3.1.2.
72%
#451 build(deps): bump pydantic to v2 craft-providers merged Merged PR #451 upgraded craft-providers to Pydantic v2 for 2.0; current project metadata requires pydantic~=2.4.
71%
#1882 build(deps): update dependency craft-cli to v2.7.0 (hotfix/3.2) charmcraft merged Merged PR #1882 as commit c6d2a2e6, updating hotfix/3.2's craft-cli dependency from 2.6.0 to 2.7.0 in two files.
71%
#838 build(deps): update dependencies rockcraft merged Merged PR #838, updating 33 pinned dependencies, including craft-parts 2.7.0 and setuptools 77.0.1; changes landed in commit d8996ccd.
71%
#1879 build(deps): update dependency craft-cli to v2.7.0 (main) charmcraft merged Renovate dependency bump merged: craft-cli pinned from ==2.6.0 to ==2.7.0 on charmcraft main, merged as commit 72c1d435 (PR #1879), 2 files changed (+2/-2) after 2 approvals.
71%
#390 build(deps): loosen and update dev dependencies craft-cli closed Merged canonical/craft-cli#390, relaxing development dependency constraints and refreshing uv.lock; it supersedes #389.
70%
#5021 build(deps): update dependency craft-cli to v2.7.0 (main) snapcraft merged Merged #5021 via commit 0d06f8006, upgrading craft-cli from 2.6.0 to 2.7.0 on main.
70%