Skip to content

refactor: Field class independent of Metadata#1751

Draft
pierrecamilleri wants to merge 5 commits into
mainfrom
refactor/field-no-metadata
Draft

refactor: Field class independent of Metadata#1751
pierrecamilleri wants to merge 5 commits into
mainfrom
refactor/field-no-metadata

Conversation

@pierrecamilleri

@pierrecamilleri pierrecamilleri commented Jul 15, 2025

Copy link
Copy Markdown
Collaborator
  • fixes #

First attempt to manage the technical debt of the Metedata class.

The aim of this PR is to properly define a descriptor class for the "Field" class, using Pydantic Models.
This descriptor should be immutable.

The first step is to introduce this descriptor for the "BooleanField".

The effort and number of changes this requires testifies of the high coupling the Metadata class has introduced over the whole codebase.

pierrecamilleri and others added 5 commits June 4, 2026 13:05
Integrated pydantic logic to IntegerField.
All tests pass (apart from 5 console ones not related to modifications)
Fixed a few issued:
- examples should enable Any type, not only strings
- descriptor was broken during the merge of two kinds
@pierrecamilleri pierrecamilleri force-pushed the refactor/field-no-metadata branch from f2b8ecc to f81ce44 Compare June 4, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants