Skip to content

Conversation

steven10a
Copy link
Collaborator

  • Added doc strings to chat.py which was missing some
  • Some reformatting from running ruff format

Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds missing docstrings to the chat.py module to comply with documentation standards and includes automatic code formatting changes from running ruff format.

Key changes:

  • Added comprehensive docstrings to all classes and methods in chat.py
  • Removed docstring-related lint rule exceptions from pyproject.toml to enforce documentation standards
  • Applied automatic code formatting that condenses multi-line function signatures and method calls

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
src/guardrails/resources/chat/chat.py Added docstrings to Chat, AsyncChat, ChatCompletions, and AsyncChatCompletions classes and their methods
src/guardrails/resources/responses/responses.py Code formatting changes including condensed function signatures and method calls
pyproject.toml Removed docstring-related lint exceptions to enforce documentation requirements

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@gabor-openai gabor-openai left a comment

Choose a reason for hiding this comment

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

TY!

@gabor-openai gabor-openai merged commit c2c7191 into main Oct 4, 2025
3 checks passed
@steven10a steven10a deleted the dev/steven/doc_strings branch October 6, 2025 14:20
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