Call `AddProblemDetailsConventions` directly on `IServiceCollection`

To use AddProblemDetailsConventions() we must do this:

services                                // returns IServiceCollection
  .AddControllers()                     // returns IMvcBuilder        <---- here's the problem
  .AddProblemDetailsConventions();

In some environments, that IMvcBuilder isn’t available or easy to access. For example when using a modular startup, or Orchard Core. Other libraries also had this problem, it’s quite common.

All that extension method does is call IMvcBuilder.IServiceCollection. Would it be possible to add an overload (not a breaking change!) to accept that directly:

public static IServiceCollection AddProblemDetailsConventions(this IServiceCollection services)
{
  //identical to other overload
}

We could call it like so:

services.AddProblemDetailsConventions();

I would patch this myself but I never figured out the github PR thing. 😃 It would be a great improvement and add major flexibility when configuring app startup.

I hope this can be considered. Thank you!

[lonix1]

Awww heck, I messed it up again 🤡 I think my PR was applied to my fork and didn’t get to this repo. Oh well I tried. It’s here BTW.

[lonix1]

Thanks for the new version! 😄