Skip to content

Use click instead of argparse on PydanticAI CLI #3080

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Use click instead of argparse on PydanticAI CLI #3080

wants to merge 15 commits into from

Conversation

@assadyousuf
Copy link
Contributor

@assadyousuf assadyousuf commented Oct 3, 2025 •
edited by DouweM
Loading

Closes #1702

@DouweM DouweM changed the title Solves #1702 Use click instead of argparse on PydanticAI CLI Oct 3, 2025
@assadyousuf
Copy link
Contributor Author

assadyousuf commented Oct 3, 2025 •
edited
Loading

@DouweM Unrelated linting errors to this PR still seem to be showing up. Thought it was an issue with my rebase in my closed PR

@assadyousuf assadyousuf mentioned this pull request Oct 3, 2025
Closed
@DouweM
Copy link
Collaborator

DouweM commented Oct 3, 2025

@assadyousuf Looking it uv.lock, it seems like you inadvertently updated the modal package. Can you roll that back and add just click?

@assadyousuf
Copy link
Contributor Author

Everything is ready on this PR now

@DouweM
Copy link
Collaborator

DouweM commented Oct 8, 2025

@Kludex Can you please check this out as you originally created #1702?

@DouweM DouweM requested a review from Kludex October 8, 2025 07:12
Kludex
Kludex reviewed Oct 10, 2025
View reviewed changes
@assadyousuf assadyousuf requested a review from Kludex October 14, 2025 00:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Kludex Kludex Awaiting requested review from Kludex

Assignees

@Kludex Kludex

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Use click instead of argparse on PydanticAI CLI

3 participants

@assadyousuf @DouweM @Kludex