← Back to issue list

build(deps): drop unnecessary pydantic-yaml

View original Github issue

Metadata

Project
craft-parts
Number
#811
Type
pull request
State
merged
Author
tigarmo
Labels
—
Created
Updated
Closed

Current evaluation

Merged #811 in commit 1debce16, removing pydantic-yaml because Pydantic 2 models are serialized with model_dump() and PyYAML directly.

Resolution: closed resolved

Reason: The dependency cleanup was merged through PR #811 as commit 1debce16, eliminating the obsolete pydantic-yaml package.

Evaluated by: mr-cal-bot

No scores available.

Issue body

Not needed anymore since Pydantic 2, where we model_dump() the model and then use PyYAML directly. - [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? -----

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged #811 in commit 1debce16, removing pydantic-yaml because Pydantic 2 models are serialized with model_dump() and PyYAML directly.
stealth/space-bunny-alpha — closed resolved Merged PR #811 in commit 1debce16, removing the unnecessary pydantic-yaml dependency; Pydantic 2 uses model_dump() with PyYAML directly.
qwen/qwen3.6-35b-a3b — — Merged to remove the pydantic-yaml dependency, as Pydantic 2 uses model_dump() with PyYAML directly. Approved by two reviewers and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 — — Merged removal of the pydantic-yaml dependency. Pydantic 2 natively handles model serialization via model_dump() and PyYAML, making the external package obsolete.

Update history

No update history recorded yet.

Related work

  • Related To: canonical/craft-parts#771 (confidence 86%)

    The earlier Pydantic 2 migration enabled replacing pydantic-yaml with model_dump() and direct PyYAML serialization.

Related issues

Issue Project State Summary Similarity
#841 Incorrect extra set on pydantic-yaml craft-parts closed Fixed by merged PR #811 (commit 1debce16), which removed the unnecessary pydantic-yaml dependency and its nonexistent pyyaml extra.
74%
#332 build(deps): drop pyyaml dependency craft-cli merged Merged PR #332 (commit 790e307) removed the unused PyYAML dependency from craft-cli's packaging metadata ahead of the planned major release.
73%
#519 deps: freeze pydantic-yaml <1.0.0 (#498) craft-parts merged Merged PR #519 as cf61a4f6, cherry-picking 5b463227 from #498 to cap pydantic-yaml below 1.0.0 and avoid breaking API changes pending refactoring.
72%
#158 ImportError: cannot import name 'YamlModel' from 'pydantic_yaml' craft-parts closed Fixed by merged PR #159, which pinned pydantic-yaml 0.4.2 in setup.py; implementation commit 0226365f, merge commit 9b90997e.
72%
#49 deps: freeze pydantic-yaml < 1.0 craft-application merged Merged PR #49 as commit 97c620ab, capping pydantic-yaml below 1.0 to avoid potentially breaking dependency releases.
72%
#193 build(deps)!: update to pydantic 2 craft-store merged Merged the breaking Pydantic 2 upgrade in #193 (commit 026b78d), updating dependencies and model code across 21 files (+135/-90).
72%
#5488 build: remove unnecessary constraints snapcraft merged Merged #5488 in commit 3d6c39f0d, removing three obsolete build dependency constraints after PyYAML 6.0.1 gained proper Cython 3 support.
71%
#498 deps: freeze pydantic-yaml <1.0.0 craft-parts merged Merged PR #498 as 5b463227, capping pydantic-yaml below 1.0.0 to avoid breaking API changes pending the refactor tracked in #499.
71%