Skip to content

Fix typos in docstrings across multiple files - #101

Merged
hangfei merged 4 commits into
google:mainfrom
fyhe8082:main
Apr 12, 2025
Merged

hangfei merged 4 commits into
google:mainfrom
fyhe8082:main

Conversation

@fyhe8082

Copy link
Copy Markdown
Contributor

reviewed all the files under the src/google/adk/agents directory and corrected the following spelling errors:

In base_agent.py, changed "conversaction" to "conversation"

In invocation_context.py, changed "conversaction" to "conversation"

In remote_agent.py, changed "dsiabled" to "disabled"

In run_config.py, changed "its" to "it's"

…nversation' and 'dsiabled' -> 'disabled'. Update comments for clarity.
@google-cla

google-cla Bot commented Apr 11, 2025

Copy link
Copy Markdown

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.

fyhe8082 and others added 3 commits April 11, 2025 16:53
- Corrected "reqeust" to "request" in auth_preprocessor.py
- Fixed "credentails" to "credentials" in auth_tool.py
- Updated "agetn" to "agent" in cli_eval.py
- Changed "creiteria" to "criteria" in cli_tools_click.py
@hangfei
hangfei merged commit ee2a753 into google:main Apr 12, 2025
fangyh20 added a commit to fangyh20/adk-python that referenced this pull request Apr 12, 2025
…le#101)

Corrected 'resposnes' to 'responses', 'uncertainity' to 'uncertainty', 'conversaction' to 'conversation', and 'exeuction' to 'execution' in relevant docstrings for clarity.
fangyh20 added a commit to fangyh20/adk-python that referenced this pull request Apr 12, 2025
…le#101)

Corrected 'resposnes' to 'responses', 'uncertainity' to 'uncertainty', 'conversaction' to 'conversation', and 'exeuction' to 'execution' in relevant docstrings for clarity.
hangfei added a commit that referenced this pull request Apr 12, 2025
#121)

* Fix typos in docstrings of evaluation_generator.py and event.py (#101)

Corrected 'resposnes' to 'responses', 'uncertainity' to 'uncertainty', 'conversaction' to 'conversation', and 'exeuction' to 'execution' in relevant docstrings for clarity.

* Fix typos in docstrings and comments across multiple files

Corrected 'detla' to 'delta', 'buil-in' to 'built-in', 'walkaround' to 'workaround', and 'conversaction' to 'conversation' for clarity in relevant files. Updated comments for consistency.

---------

Co-authored-by: Hangfei Lin <hangfei@google.com>
coder-aditi pushed a commit to coder-aditi/adk-python that referenced this pull request Jul 2, 2025
* Fix typos in docstrings across multiple files: 'conversaction' -> 'conversation' and 'dsiabled' -> 'disabled'. Update comments for clarity.

* Fix typos in comments and docstrings across multiple files

- Corrected "reqeust" to "request" in auth_preprocessor.py
- Fixed "credentails" to "credentials" in auth_tool.py
- Updated "agetn" to "agent" in cli_eval.py
- Changed "creiteria" to "criteria" in cli_tools_click.py

---------

Co-authored-by: fangyu <fangyu.jobs@gmail.com>
Co-authored-by: Hangfei Lin <hangfei@google.com>
coder-aditi pushed a commit to coder-aditi/adk-python that referenced this pull request Jul 2, 2025
…le#101) (google#121)

* Fix typos in docstrings of evaluation_generator.py and event.py (google#101)

Corrected 'resposnes' to 'responses', 'uncertainity' to 'uncertainty', 'conversaction' to 'conversation', and 'exeuction' to 'execution' in relevant docstrings for clarity.

* Fix typos in docstrings and comments across multiple files

Corrected 'detla' to 'delta', 'buil-in' to 'built-in', 'walkaround' to 'workaround', and 'conversaction' to 'conversation' for clarity in relevant files. Updated comments for consistency.

---------

Co-authored-by: Hangfei Lin <hangfei@google.com>
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