do we need Pydantic?
Metadata
Current evaluation
Unused Pydantic dependency was removed by merged PR #207 (commit ce2ee08), avoiding a Pydantic 2 migration.
Resolution: closed resolved
Reason: PR #207 removed Pydantic from three build-configuration files and was merged as commit ce2ee08, resolving #201 without requiring a Pydantic 2 migration.
Evaluated by: mr-cal-bot
No scores available.
Issue body
### What needs to get done
We currently have a dependency on Pydantic, but as far as I can tell we don't use the lib at all. Check if we can drop the dependency.
### Why it needs to get done
Migrating to Pydantic 2 is not a trivial task, and can be totally avoided if we can drop the dep altogether
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Unused Pydantic dependency was removed by merged PR #207 (commit ce2ee08), avoiding a Pydantic 2 migration. | |
| stealth/space-bunny-alpha | — | closed resolved | Pydantic was confirmed unused and removed by merged PR #207 (commit ce2ee08), avoiding a Pydantic 2 migration. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution or comments. The proposal to drop the unused Pydantic dependency and avoid a Pydantic 2 migration was abandoned. | |
| qwen/qwen3.6-35b-a3b | — | — | Abandoned; closed without activity or resolution, leaving the unused Pydantic dependency status unverified. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without comments or resolution. The request to remove the unused Pydantic dependency was likely abandoned or resolved by dropping the package to avoid migration overhead. | |
| qwen/qwen3.6-35b-a3b | — | — | Closed without resolution. The Pydantic dependency review was abandoned with zero comments, leaving the library and migration concern unaddressed. |
Update history
No update history recorded yet.
Related work
-
Likely Fixed By:
canonical/craft-cli#207
(confidence 100%)
Merged PR #207 directly removed the unused dependency and closed the issue.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #347 requirements: freeze pydantic<2.0 | craft-providers | merged | Merged #347 (commit ee9f32d), pinning pydantic<2.0 to avoid test failures caused by Pydantic v2 incompatibilities. | |
| #582 [EXPERIMENTAL] build(deps): use pydantic 2 | craft-parts | closed | Closed unmerged in Aug 2024 by @lengau as superseded; the Pydantic 2 migration landed later via merged PR #771 (commit 6b155294) plus follow-ups #776, #778, #794, #811. |