Skip to content

Fix/gr regression fixes post pydantic v2 release#116

Merged
stevenhsd merged 2 commits into
release_v09from
fix/gr-regression_fixes_post_pydantic_v2_release
Jun 23, 2026
Merged

Fix/gr regression fixes post pydantic v2 release#116
stevenhsd merged 2 commits into
release_v09from
fix/gr-regression_fixes_post_pydantic_v2_release

Conversation

@georgeRobertson

Copy link
Copy Markdown
Contributor

TLDR of changes

  • fix is complex type to be able to decipher a type from a custom object/type
  • fix mandatory fields logic where type is defined within the dischema

What kind of changes does this PR introduce?

Tick all that apply

  • fix: A bug fix. Correlates with PATCH in SemVer
  • feat: A new feature. Correlates with MINOR in SemVer
  • docs: Documentation only changes
  • style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
  • refactor: A code change that neither fixes a bug nor adds a feature
  • perf: A code change that improves performance
  • test: Adding missing or correcting existing tests
  • build: Changes that affect the build system or external dependencies (example scopes: pip, docker, npm)
  • ci: Changes to CI configuration files and scripts (example scopes: GitLabCI)

Please check if the PR fulfills these requirements

  • I have read and followed the Contributing guidance
  • Docs have been added / updated
  • Tests and Linting in the CI are passing
  • Changes have been reviewed and approved by a Project Maintainer

…of 'simple' types

added some more tests around is complex and stringify model to ensure functionality is working
This was causing custom type objects in the dischema to become unmandatory despite being lists in the mandatory_fields section
@stevenhsd stevenhsd merged commit bb3d1c7 into release_v09 Jun 23, 2026
4 checks passed
@sonarqubecloud

Copy link
Copy Markdown

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