-
Notifications
You must be signed in to change notification settings - Fork 25.1k
Do not show unnecessary steps for creating AOT-compatible OpenAPI application when running in .NET 10 #34757
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
Conversation
…nning .NET 9 or lower I also removed the --prerelease flag from the package, no reason to add a prerelease version; AOT works fine without prerelease
|
Currently I've added I've looked through several commits to find out how I could set it to this preview-2 value, but I can't figure this out. If there's any changes required, let me know! |
|
Important note: This should probably only get merged when dotnet/aspnetcore#60337 is merged. |
Rick-Anderson
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:::moniker range="= aspnetcore-9.0" is preferred over <10
|
Hi @Rick-Anderson ! The main PR has been merged: dotnet/aspnetcore#60337 I've changed this PR's title and draft status, I believe it can be merged :)? |
|
@sander1095 thanks so much for your contribution. |
I have also removed the
--previewflag as it seems unnecessary. It's not necessary to add a preview package to get OpenAPI to be AOT compatible in .NET 9.Fixes #34756
Internal previews