David Acker
David Acker
We'll want to report this diagnostic for `AddMvc` as well, since it calls `AddControllersWithViews` and `AddRazorPages` internally.
# API for proposed analyzer ## Diagnostic: **Title:** Custom IProblemDetailsWriter is incorrectly configured **Message:** The custom IProblemDetailsWriter must be registered before calling AddControllers, AddControllersWithViews, AddMvc, or AddRazorPages. **Category:** Usage **Severity:**...
@halter73 I can add a code fix for moving the `IProblemDetailsWriter` registration up. Should we add the code fixes to the existing analyzers project, like we do with Mvc.Api.Analyzers, or...
Sounds good! I'll move the new analyzer over to `AspNetCoreAnalyzers` in the PR that I have up. Originally, I added it to the legacy analyzers project in order to leverage...
I updated the failing test to be skipped on Linux and macOS. I also reverted the last commit which added the `.editorconfig` file.