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

Update simple_type.py #1247

Merged
merged 1 commit into from
Dec 16, 2024
Merged

Update simple_type.py #1247

merged 1 commit into from
Dec 16, 2024

Conversation

stevenbedrick
Copy link
Contributor

@stevenbedrick stevenbedrick commented Dec 10, 2024

Fixed minor typo in documentation description template for JSON-schema- "formated" should be "formatted".


Important

Fix typo in __class_getitem__() docstring in simple_type.py.

  • Documentation:
    • Fix typo in __class_getitem__() docstring in simple_type.py, changing "Formated" to "Formatted".

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

Fixed minor typo in documentation description template for JSON-schema- "formated" should be "formatted".
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 e4c8c56 in 7 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. instructor/dsl/simple_type.py:29
  • Draft comment:
    Typo fix is correct. No further issues related to this change.
  • Reason this comment was not posted:
    Confidence changes required: 0%
    The PR correctly fixes a typo in the documentation string. There are no other issues related to this change.

Workflow ID: wflow_yKM5IWWv7qIeGVU4


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

@jxnl jxnl merged commit a953b99 into instructor-ai:main Dec 16, 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