Skip to content

Conversation

@cleemullins
Copy link
Collaborator

More doc reference fixes

Copy link
Contributor

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 enhances type annotations in various modules across the codebase by updating docstring type references to use full class paths and adding missing documentation for methods and parameters. The changes improve clarity and consistency in the documentation.

  • Updated type annotations from simple class names to full class path references using the :class: directive
  • Added comprehensive docstrings for previously undocumented methods and parameters
  • Standardized documentation format across multiple modules

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
channel_service_adapter.py Added docstrings for multiple methods and updated type references to use full class paths
state.py Enhanced documentation with complete parameter and return type annotations
authorization.py Updated type references and added missing docstrings for private methods
agent_application.py Added docstrings for properties and methods with full type annotations
activity_handler.py Fixed parameter names and updated type references to use full class paths
msal_connection_manager.py Enhanced method documentation with complete type annotations
semantic_action.py Corrected class name reference in type annotation

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

@cleemullins cleemullins merged commit f6913e3 into main Oct 10, 2025
8 checks passed
@cleemullins cleemullins deleted the users/chmullins/moreDocsCleanup branch October 10, 2025 21:42
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