Skip to content

Conversation

satawatnack
Copy link
Member

No description provided.

README.md Outdated

## Requirements

- **Python:** >=3.12,<3.13
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we target lower version of Python? requests supports Python 3.8+. tavily-python supports Python 3.6+

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I use 3.10
due to match/case statement, | None (introduced in Python 3.10)

@satawatnack satawatnack requested a review from sorawit July 25, 2025 08:40
Copy link
Member

@sorawit sorawit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I prefer we define specific type for response (maybe use Pydantic) instead of using generic dict, but that can be done in other PRs.

@taobun taobun merged commit c1fa0c0 into main Jul 31, 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.

4 participants