Skip to content

Conversation

@mikekistler
Copy link
Contributor

Update prompt for minimal APIs + OpenAPI for the OpenAPI document generation feature in .NET 9.

Copilot AI review requested due to automatic review settings June 26, 2025 23:19
@mikekistler mikekistler marked this pull request as draft June 26, 2025 23:19

This comment was marked as outdated.

@aaronpowell
Copy link
Contributor

@mikekistler is there anything outstanding on this PR before merging?

@mikekistler mikekistler marked this pull request as ready for review July 2, 2025 14:24
@mikekistler
Copy link
Contributor Author

@aaronpowell Okay ... ready now. I reviewed with @captainsafia offline and pushed one change she recommended.

@aaronpowell aaronpowell requested a review from Copilot July 3, 2025 01:30
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

Updates the ASP.NET Minimal API prompt to incorporate new .NET 9 OpenAPI document generation features.

  • Adds guidance for standard error responses using ProblemDetailsService and StatusCodePages
  • Refactors the OpenAPI section title and introduces built-in .NET 9 document support, operationId naming, property descriptions, and transformers
  • Removes the old “/ Swagger” title and consolidates related documentation items into the updated section
Comments suppressed due to low confidence (1)

prompts/aspnet-minimal-api-openapi.prompt.md:1

  • Please update the README.md to include the new .NET 9 OpenAPI document generation features introduced in this PR, ensuring documentation stays in sync.
---

Copy link
Contributor

@jamesmontemagno jamesmontemagno left a comment

Choose a reason for hiding this comment

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

@aaronpowell good to go

@aaronpowell aaronpowell merged commit 99fb7e1 into github:main Jul 3, 2025
1 check passed
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