Skip to content
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

chore: update Tools.py #546

Closed
wants to merge 1 commit into from
Closed

chore: update Tools.py #546

wants to merge 1 commit into from

Conversation

eltociear
Copy link
Contributor

@eltociear eltociear commented Oct 2, 2024

overriden -> overridden


Important

Fix typo in FunctionDef class docstring in Tools.py.

  • Typo Fix:
    • Corrected typo in FunctionDef class docstring: "overriden" to "overridden" in Tools.py.

This description was created by Ellipsis for 85cc0ad. It will automatically update as commits are pushed.

overriden -> overridden
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

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

👍 Looks good to me! Reviewed everything up to 85cc0ad in 5 seconds

More details
  • Looked at 13 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 1 drafted comments based on config settings.
1. agents-api/agents_api/autogen/Tools.py:72
  • Draft comment:
    Typo fix: 'overriden' corrected to 'overridden'.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The PR correctly fixes a typo in the comment, changing 'overriden' to 'overridden'.

Workflow ID: wflow_MahfmbOeCjAq3sq3


You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

@creatorrr
Copy link
Contributor

@eltociear - the Tool.py file is actually autogenerated. Can you make the fix inside https://github.com/julep-ai/julep/blob/dev/typespec/tools/models.tsp#L34 ?

@eltociear
Copy link
Contributor Author

@creatorrr Thanks!
I opened a new PR.
#549

@creatorrr creatorrr closed this Oct 2, 2024
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.

2 participants