Skip to content

Remove unused pydantic#146

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

Remove unused pydantic#146
ka-sarthak merged 1 commit intoFAIRmat-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
Copy link
Copy Markdown
Contributor Author

DanielYang59 commented Aug 11, 2025

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