Skip to content

Conversation

@Quantisan
Copy link
Contributor

correcting MCP terminology in a docstring to avoid confusion

@google-cla
Copy link

google-cla bot commented Apr 22, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

Copy link
Collaborator

@hangfei hangfei left a comment

Choose a reason for hiding this comment

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

please fix commit message(see check failures)

This function transforms an ADK tool definition into its equivalent
representation in the MCP (Model Control Plane) system.
representation in the MCP (Model Context Protocol) system.
Copy link
Collaborator

Choose a reason for hiding this comment

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

-> MCP (Model Context Protocol) representation

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Quantisan please change to this for brevity.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Copy link
Collaborator

Choose a reason for hiding this comment

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

Thanks. But Let's keep the original indentation.

@Quantisan
Copy link
Contributor Author

updated commit message

@Quantisan Quantisan requested a review from hangfei April 23, 2025 03:54
@Quantisan Quantisan force-pushed the patch-2 branch 2 times, most recently from 7e45607 to 38ce9d1 Compare April 23, 2025 04:22
@Quantisan Quantisan changed the title fix typo in conversion_utils.py docs: correct typo in conversion_utils.py Apr 23, 2025
@Quantisan Quantisan force-pushed the patch-2 branch 2 times, most recently from 379d475 to 6bdd19f Compare April 23, 2025 07:58
@hangfei
Copy link
Collaborator

hangfei commented Apr 28, 2025

Could you use original indentation?

@Quantisan
Copy link
Contributor Author

Could you use original indentation?

updated

Copy link
Collaborator

@hangfei hangfei left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@hangfei hangfei merged commit 4ae8d72 into google:main Apr 29, 2025
6 checks passed
@hangfei hangfei added the ready to pull [Status] This PR is ready to be importing back to Google label Apr 29, 2025
@Quantisan Quantisan deleted the patch-2 branch April 30, 2025 06:00
coder-aditi pushed a commit to coder-aditi/adk-python that referenced this pull request Jul 2, 2025
* docs: correct typo in conversion_utils.py

* docs: edited for conciseness
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to pull [Status] This PR is ready to be importing back to Google

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants