Skip to content

Remove unused pydantic#146

Merged
ka-sarthak merged 1 commit into
FAIRmat-NFDI:mainfrom
DanielYang59:remove-pydantic
Aug 11, 2025
Merged

Remove unused pydantic#146
ka-sarthak merged 1 commit into
FAIRmat-NFDI:mainfrom
DanielYang59:remove-pydantic

Conversation

@DanielYang59

Copy link
Copy Markdown
Contributor

No description provided.

@ka-sarthak

Copy link
Copy Markdown
Collaborator

The constraint was added to avoid a pre-release version of Pydantic #140. Is this not an issue anymore?

@DanielYang59

DanielYang59 commented Aug 11, 2025

Copy link
Copy Markdown
Contributor Author

I don't think pre-release would be considered by default? @blueraft also I don't see pydantic being used in the code base?

@blueraft

Copy link
Copy Markdown
Contributor

That issue is fixed now!

@ka-sarthak

Copy link
Copy Markdown
Collaborator

Great! Then the changes look good. Thanks @DanielYang59!

@ka-sarthak ka-sarthak merged commit b623769 into FAIRmat-NFDI:main Aug 11, 2025
5 checks passed
@DanielYang59 DanielYang59 deleted the remove-pydantic branch August 11, 2025 12:37
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.

3 participants