Skip to content

Fix master required pydantic to be installed#329

Closed
DonutByte wants to merge 1 commit into
prkumar:masterfrom
DonutByte:fix/optional-dependency-pydantic-required
Closed

Fix master required pydantic to be installed#329
DonutByte wants to merge 1 commit into
prkumar:masterfrom
DonutByte:fix/optional-dependency-pydantic-required

Conversation

@DonutByte

Copy link
Copy Markdown

Fixes #328

Changes proposed in this pull request:

  • Remove line which caused pydantic to be required

Attention: @prkumar

@prkumar prkumar requested review from leiserfg and prkumar April 26, 2025 21:42
@leiserfg

Copy link
Copy Markdown
Collaborator

Solved by #332

@leiserfg

Copy link
Copy Markdown
Collaborator

Thanks for the pointer to the issue @DonutByte

@DonutByte

Copy link
Copy Markdown
Author

Seems like #332 fixes it better 😅
So I will close this PR...

@DonutByte DonutByte closed this Apr 28, 2025
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.

Importing uplink from master fails without pydantic

2 participants