fix: fix pyright unnecessary type ignore comment

[ivanleomk]

Important

Remove unnecessary # type: ignore comments from pydantic imports and adjust formatting in several files.

• Type Ignore Removal:
  • Removed # type: ignore from pydantic imports in batch.py, citation.py, iterable.py, maybe.py, validators.py, and multimodal.py.
• Formatting:
  • Added trailing commas in dictionary definitions in batch.py and iterable.py.
  • Removed unnecessary line breaks in multimodal.py.

^{This description was created by for b07286c. It will automatically update as commits are pushed.}

[cloudflare-workers-and-pages]

Deploying instructor-py with    Cloudflare Pages

Latest commit:	b07286c
Status:	✅  Deploy successful!
Preview URL:	https://6336b970.instructor-py.pages.dev
Branch Preview URL:	https://fix-pyright-ignore-errors.instructor-py.pages.dev

View logs

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to b07286c in 2 minutes and 16 seconds

More details

• Looked at 169 lines of code in 6 files
• Skipped 0 files when reviewing.
• Skipped posting 4 drafted comments based on config settings.

1. instructor/batch.py:6

• Draft comment:
  Unnecessary trailing comma in the dictionary definition. Remove it for cleaner code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR removes unnecessary type ignore comments for pydantic imports. This is a good change as it cleans up the code. However, there are some formatting issues that need to be addressed.

2. instructor/dsl/iterable.py:115

• Draft comment:
  Unnecessary trailing comma in the set definition. Remove it for cleaner code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR removes unnecessary type ignore comments for pydantic imports. This is a good change as it cleans up the code. However, there are some formatting issues that need to be addressed.

3. instructor/multimodal.py:343

• Draft comment:
  Unnecessary trailing comma in the list definition. Remove it for cleaner code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The PR removes unnecessary type ignore comments for pydantic imports. This is a good change as it cleans up the code. However, there are some formatting issues that need to be addressed.

4. instructor/batch.py:3

• Draft comment:
  Since the pydantic import has been modified, please ensure that the relevant documentation and tests are updated accordingly.
• Reason this comment was not posted:
  Comment did not seem useful.

Workflow ID: wflow_NgJqD43tKNjRP2CH

---

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