-
Notifications
You must be signed in to change notification settings - Fork 10.3k
Minimal APIs naming cleanup #35478
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
Comments
Thank you for submitting this for API review. This will be reviewed by @dotnet/aspnet-api-review at the next meeting of the ASP.NET Core API Review group. Please ensure you take a look at the API review process documentation and ensure that:
|
Thank you for submitting this for API review. This will be reviewed by @dotnet/aspnet-api-review at the next meeting of the ASP.NET Core API Review group. Please ensure you take a look at the API review process documentation and ensure that:
|
Thank you for submitting this for API review. This will be reviewed by @dotnet/aspnet-api-review at the next meeting of the ASP.NET Core API Review group. Please ensure you take a look at the API review process documentation and ensure that:
|
1 similar comment
Thank you for submitting this for API review. This will be reviewed by @dotnet/aspnet-api-review at the next meeting of the ASP.NET Core API Review group. Please ensure you take a look at the API review process documentation and ensure that:
|
The rename was merged into rc1 by #35632. |
We should not use "Minimal" in any API names. We should also avoid the word "action" when defining minimal route handlers.
MinimalActionEndpointRouteBuilderExtensions
->DelegateEndpointRouteBuilderExtensions
MinimalActionEndpointConventionBuilder
->DelegateEndpointConventionBuilder
Map(this IEndpointRouteBuilder endpoints, RoutePattern pattern, Delegate action)
->Map(this IEndpointRouteBuilder endpoints, RoutePattern pattern, Delegate handler)
RequestDelegateFactory.Create(Delegate action, RequestDelegateFactoryOptions? options)
->RequestDelegateFactory.Create(Delegate handler, RequestDelegateFactoryOptions? options)
The text was updated successfully, but these errors were encountered: