Skip to content
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

include AOT support in v9 moniker #34613

Merged
merged 2 commits into from
Feb 3, 2025
Merged

Conversation

timdeschryver
Copy link
Contributor

@timdeschryver timdeschryver commented Jan 31, 2025

Closes #34510

As the issue points out the section about Native AOT is also showing on < aspnetcore-9.0 pages (at the top of the page).
This PR moves the section within the >= aspnetcore-9.0 moniker range, as it was released in v9.0


Internal previews

📄 File 🔗 Preview link
aspnetcore/fundamentals/openapi/aspnetcore-openapi.md Generate OpenAPI documents

<!--keep-->[AddServiceDefaults](https://source.dot.net/#TestingAppHost1.ServiceDefaults/Extensions.cs,0f0d863053754768,references) Adds common .NET Aspire services such as service discovery, resilience, health checks, and OpenTelemetry.

:::moniker-end
[AddServiceDefaults](https://source.dot.net/#TestingAppHost1.ServiceDefaults/Extensions.cs,0f0d863053754768,references)<!--keep--> adds common .NET Aspire services such as service discovery, resilience, health checks, and OpenTelemetry.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Additionally, I noticed a mall formatted link at https://learn.microsoft.com/en-us/aspnet/core/fundamentals/openapi/aspnetcore-openapi?view=aspnetcore-9.0&tabs=visual-studio#customizing-run-time-behavior-during-build-time-document-generation.

image

Due to <!--keep-->, the links is not detected. AFAIK this happens because the comment is the first word on this line. That's why I moved the keep comment after the link.
This change shouldn't interfere with #34489.

cc @Rick-Anderson

@Rick-Anderson Rick-Anderson merged commit da35f22 into dotnet:main Feb 3, 2025
3 checks passed
@Rick-Anderson
Copy link
Contributor

@timdeschryver THANKS!

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.

OpenAPI's AOT support is mentioned for a too-wide range of framework versions
2 participants