← 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 to remove the pydantic-yaml dependency, as Pydantic 2 uses model_dump() with PyYAML directly. Approved by two reviewers and passed CI checks.

Suggested action:

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
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 issues

Issue Project State Summary Similarity
#332 build(deps): drop pyyaml dependency craft-cli merged Merged to drop the unused pyyaml dependency ahead of a major release. Approved by two reviewers, passed CI checks, and modified a single file.
82%
#49 deps: freeze pydantic-yaml < 1.0 craft-application merged Merged into main after two approvals and passing CI. The change pins pydantic-yaml to versions below 1.0 to prevent breaking changes. Code coverage remains unchanged.
74%
#498 deps: freeze pydantic-yaml <1.0.0 craft-parts merged Merged a dependency update freezing pydantic-yaml below 1.0.0 to avoid API breaking changes until refactoring is complete. Approved by one reviewer and passed CI checks.
73%
#519 deps: freeze pydantic-yaml <1.0.0 (#498) craft-parts merged Merged a dependency freeze pinning pydantic-yaml below version 1.0.0 to prevent breaking API changes that require future refactoring. Cherry-picked, approved, passed CI, and modified one file.
72%
#193 build(deps)!: update to pydantic 2 craft-store merged Merged to upgrade the project to pydantic 2. Approved by two reviewers, the update modified 21 files with 135 additions and 90 deletions.
72%
#5488 build: remove unnecessary constraints snapcraft merged Merged PR removes unnecessary dependency constraints from pyproject.toml. Updated after a reviewer noted PyYAML 6.0.1 natively supports Cython 3. Approved by two reviewers with all CI checks passing.
71%
#5173 build(deps): update pydantic etc. (main) snapcraft merged Merged automated dependency update upgrading pydantic to 2.10.3 and pydantic-core to 2.27.1. Approved by two reviewers, passed CI, and merged into main.
71%
#419 deps: loosen pydantic dependency craft-parts merged Merged after approval and passing CI. Loosened the pydantic dependency constraint, removing the previous freeze at version 1.9 now that the upstream pydantic-yaml compatibility issue is resolved.
70%