Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Dec 18, 2025

Removes the migration guide for transitioning from get_tools() to fetch_tools(). This section is no longer relevant since get_tools() was completely removed in #42.


Summary by cubic

Removed the outdated migration section for get_tools() → fetch_tools() from the README. This cleans up the docs and avoids confusion now that get_tools() is fully removed.

Written for commit 784188c. Summary will update automatically on new commits.

Removes the migration guide for get_tools() to fetch_tools() transition.
This section is no longer relevant since get_tools() was completely
removed in favour of fetch_tools() (commit d50d5fb).
Copilot AI review requested due to automatic review settings December 18, 2025 19:16
@ryoppippi ryoppippi enabled auto-merge (squash) December 18, 2025 19:16
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 1 file

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the obsolete migration guide section from the README that documented how to transition from the deprecated get_tools() method to fetch_tools(). Since get_tools() was completely removed in a previous PR (#42), this migration guide is no longer needed.

Key Changes:

  • Removes 25 lines of documentation describing the migration path from get_tools() to fetch_tools()

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

@willleeney willleeney left a comment

Choose a reason for hiding this comment

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

lgtm

@ryoppippi ryoppippi merged commit bdcf90d into main Dec 19, 2025
13 checks passed
@ryoppippi ryoppippi deleted the chore/remove-migration-section branch December 19, 2025 12:42
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