Switch to minimal hosting and actions #2
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
martincostello
force-pushed
the
minimal-hosting
branch
2 times, most recently
from
b107d77 to
630c75b
Compare
martincostello
force-pushed
the
minimal-hosting
branch
2 times, most recently
from
0185a77 to
5173026
Compare
Closed
martincostello
force-pushed
the
minimal-hosting
branch
3 times, most recently
from
d7126c2 to
2c9bf00
Compare
martincostello
force-pushed
the
minimal-hosting
branch
5 times, most recently
from
a0b75d5 to
1493d95
Compare
Move to using WebApplicationBuilder using a daily build of .NET 6 preview 6.
Update the endpoints to use new minimal APIs.
Bind the item Ids as a Guid instead of a string.
Add workaround to bootstrap the HttpServerFixture when using top-level programs.
Add workaround for test configuration changes not being used.
Add link to PR to support ClaimsPrincipal.
When the debugger is attached, wait indefinitely for the host to start.
Explicitly configure the development environment due to issue with it not being set by default.
Simplify the bootstrapping to leverage implicit conventions.
Set the minimum logging level for the tests to debug while there's still issues with things.
Use JsonResult instead of ObjectResult.
Pre-empt changes coming in a future preview, as it reduces the amount of code in the endpoints and gets rid of the need to cast to IResult for resolving ambiguity.
Update to a daily build of .NET 6 preview 7.
Remove workaround for timeout when debugging host startup which is now fixed.
As of preview 7, the SDK now uses C# 10 by default.
Document the HTTP status codes that each API endpoint returns.
Update to latest .NET 6 daily build of the SDK. React to additional built-in global using statements.
Move some workarounds to only be for Visual Studio, rather than also on the CLI.
Update to the latest daily build of the .NET 6 preview 7 SDK.
Use Problem(...) instead of BadRequest() and NotFound().
Remove the route attributes as they Just Work anyway.
Use new() instead of explicit type.
Update to a newer version of the .NET 6 preview 7 SDK. Remove workaround for broken Swagger documentation. Add workaround for "preview features" warning.
Move CA2252 workaround to the props file instead of the code.
Remove workarounds required to build in Visual Studio that are not required with Visual Studio 2022 17.0.0 Preview 2.0.
Switch to using file-scoped namespaces.
Update to an RC1 build of the .NET SDK. Remove HACK to prevent spurious code analysis warning.
Update to a newer RC1 build that includes fix for missing endpoint metadata.
Refactor the extension methods to resemble those proposed to be added to ASP.NET Core in a future build.
Tweak the indentation etc. for the actions.
Update to a newer build of the .NET SDK.
Update to a newer .NET 6 SDK RC1 build. Remove need to explicitly configure UseDeveloperExceptionPage().
Update .NET SDK to pick up fix for issues with IConfiguration and remove workaround.
Remove unused method from HttpServerFixture.
Update to the latest daily build of the .NET 6 RC1 SDK.
martincostello
force-pushed
the
minimal-hosting
branch
from
74c18c8 to
7dee0c3
Compare
Add Microsoft.Net.Compilers.Toolset so that file-scoped namespaces compile in Visual Studio 2022 preview 2.
Update to the latest .NET 6 SDK for RC1. Remove workarounds for fixed issues.
Fix warnings in test logs about use of HTTP.
Add a new line between the namespace and the class declaration.
martincostello
added
.NET
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Refactor the application to use the new minimal hosting and actions features of ASP.NET Core 6 using a daily build of Release Candidate 1.
Various workarounds have been applied for known issues, but the test still fails due to Razor pages not working. This seems to be because the content root applied by the tests does not take effect.