Skip to content

Agents SDK API reference documentation #302

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

Merged
merged 10 commits into from
Jul 15, 2025
Merged

Agents SDK API reference documentation #302

merged 10 commits into from
Jul 15, 2025

Conversation

phecke
Copy link
Member

@phecke phecke commented May 20, 2025

  • Add namespace to ///<exception cref=""> targets so they are rendered as links in the published API reference documentation. This fixes doc build warnings.
  • Add missing API descriptions
  • Fixed example code formatting
  • Removed unneeded spaces.

@Copilot Copilot AI review requested due to automatic review settings May 20, 2025 21:47
@phecke phecke requested a review from a team as a code owner May 20, 2025 21:47
@phecke phecke self-assigned this May 20, 2025
@phecke phecke added the documentation Improvements or additions to documentation label May 20, 2025
@github-actions github-actions bot added ML: Core Tags changes to core libraries and removed documentation Improvements or additions to documentation labels May 20, 2025
Copy link
Contributor

@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 the System namespace to cref targets in XML documentation comments so that exception types are rendered as links in the published API reference documentation.

  • Updated cref attributes for exception documentation to fully qualify exception types
  • Adjusted code formatting in a couple of methods for consistency

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Verify.cs Updated cref attributes for ArgumentNullException
Record.cs Updated cref attribute for InvalidCastException and adjusted formatting of closing braces and semicolons
ActionPlanner.cs Updated cref attribute for Exception
ChatRole.cs Updated cref attribute for ArgumentNullException
AzureOpenAIModelOptions.cs Updated cref attribute for ArgumentException
IActionCollection.cs Updated cref attributes for ArgumentException
ActionEntry.cs Updated cref attribute for Exception
AI.cs Updated cref attributes for InvalidOperationException

@phecke phecke added documentation Improvements or additions to documentation HOLD draft and removed HOLD labels May 20, 2025
@phecke phecke marked this pull request as draft May 20, 2025 21:56
@phecke phecke removed the draft label May 20, 2025
@phecke phecke changed the title Added namespace to cref's Agents SDK API reference documentation May 21, 2025
@tracyboehrer tracyboehrer requested a review from goldenwitch May 23, 2025 16:42
@phecke phecke marked this pull request as ready for review July 15, 2025 01:27
@phecke phecke merged commit 6b90685 into main Jul 15, 2025
9 checks passed
@phecke phecke deleted the api-descriptions branch July 15, 2025 01:27
@phecke phecke restored the api-descriptions branch July 15, 2025 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation ML: Core Tags changes to core libraries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants