-
Notifications
You must be signed in to change notification settings - Fork 15.7k
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.
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
Community: Google Books API Tool #27307
Community: Google Books API Tool #27307
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Hi @efriis I hope you’re doing well! I was just wondering if there are any updates with regards to the status of this PR? I understand there are a lot of PRs to review, but if you have a moment to look it over or let me know if there’s anything I can do to further improve it, that would be great. Thanks! |
Hey there! You'll have to get CI passing for this to be mergeable, and we'll unfortunately close it if you can't yourself :(. Here's how you can run it locally: First, install poetry, e.g. with Then, for example if you're in community, you can run # enter package you're checking, e.g. community:
cd libs/community
# format and lint
poetry install --with lint,typing --sync
make format lint
# test
poetry install --with lint,typing,test --sync
make test
# extended tests - install extra deps with uv - not a requirement but it's how I do it
poetry install --with lint,typing,test --sync
poetry run pip install uv
poetry run uv pip install -r extended_testing_deps.txt
make extended_tests
# run a specific test - helpful if trying to just fix a single test
poetry run pytest --disable-socket --allow-unix-socket tests/unit_tests -k test_my_test_name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks! apologies for the file move impacts on load_tools
No worries, thanks again for elevating the discussion and taking the time to review the pull request! |
Description
As proposed in our earlier discussion #26977 we have introduced a Google Books API Tool that leverages the Google Books API found at https://developers.google.com/books/docs/v1/using to generate book recommendations.
Sample Usage
Sample Output
Issue
This closes #27276
Dependencies
No additional dependencies were added