Skip to content
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

Updated md table formatting of the diagnostics #35062

Merged
merged 1 commit into from
Mar 26, 2025

Conversation

benhopkinstech
Copy link
Contributor

@benhopkinstech benhopkinstech commented Mar 26, 2025

When I was writing the documentation for ASP0027 I spotted that the tables were badly formatted across the diagnostic md files.

ASP0028 had a neatly formatted grid which is more readable at first look of the file.

This PR resolves this across all the diagnostic md files.


Internal previews

Toggle expand/collapse
📄 File 🔗 Preview link
aspnetcore/diagnostics/asp0000.md ASP0000: Do not call 'IServiceCollection.BuildServiceProvider' in 'ConfigureServices'
aspnetcore/diagnostics/asp0001.md "ASP0001: Authorization middleware is incorrectly configured"
aspnetcore/diagnostics/asp0003.md ASP0003: Do not use model binding attributes with route handlers
aspnetcore/diagnostics/asp0004.md aspnetcore/diagnostics/asp0004
aspnetcore/diagnostics/asp0005.md ASP0005: Do not place attribute on method called by route handler lambda
aspnetcore/diagnostics/asp0006.md "ASP0006: Do not use non-literal sequence numbers"
aspnetcore/diagnostics/asp0007.md ASP0007: Route parameter and argument optionality is mismatched
aspnetcore/diagnostics/asp0008.md aspnetcore/diagnostics/asp0008
aspnetcore/diagnostics/asp0009.md ASP0009: Do not use Configure with WebApplicationBuilder.WebHost
aspnetcore/diagnostics/asp0010.md "ASP0010: Do not use UseStartup with WebApplicationBuilder.WebHost"
aspnetcore/diagnostics/asp0011.md ASP0011: Suggest using builder.Logging over Host.ConfigureLogging or WebHost.ConfigureLogging
aspnetcore/diagnostics/asp0012.md aspnetcore/diagnostics/asp0012
aspnetcore/diagnostics/asp0013.md ASP0013: Suggest switching from using Configure methods to WebApplicationBuilder.Configuration
aspnetcore/diagnostics/asp0014.md "ASP0014: Suggest using top level route registrations"
aspnetcore/diagnostics/asp0015.md ASP0015: Suggest using IHeaderDictionary properties
aspnetcore/diagnostics/asp0016.md aspnetcore/diagnostics/asp0016
aspnetcore/diagnostics/asp0017.md ASP0017: Invalid route pattern
aspnetcore/diagnostics/asp0018.md "ASP0018: Unused route parameter"
aspnetcore/diagnostics/asp0019.md ASP0019: Suggest using IHeaderDictionary.Append or the indexer
aspnetcore/diagnostics/asp0020.md aspnetcore/diagnostics/asp0020
aspnetcore/diagnostics/asp0021.md ASP0021: The return type of the BindAsync method must be ValueTask<T>.
aspnetcore/diagnostics/asp0022.md "ASP0022: Route conflict detected between route handlers"
aspnetcore/diagnostics/asp0023.md ASP0023: Route conflict detected between route handlers
aspnetcore/diagnostics/asp0024.md aspnetcore/diagnostics/asp0024
aspnetcore/diagnostics/asp0025.md ASP0025: Use AddAuthorizationBuilder to register authorization services and construct policies.
aspnetcore/diagnostics/asp0026.md "ASP0026: Analyzer to warn when [Authorize] is overridden by [AllowAnonymous] from 'farther away'"
aspnetcore/diagnostics/bl0001.md BL0001: Component parameter should have public setters
aspnetcore/diagnostics/bl0002.md aspnetcore/diagnostics/bl0002
aspnetcore/diagnostics/bl0003.md BL0003: Component parameter with CaptureUnmatchedValues has the wrong type
aspnetcore/diagnostics/bl0004.md "BL0004: Component parameter should be public"

Note

This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.

Copy link
Collaborator

@guardrex guardrex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @benhopkinstech! 🚀

@guardrex guardrex merged commit a14ba0d into dotnet:main Mar 26, 2025
3 checks passed
@benhopkinstech benhopkinstech deleted the diagnosticgrids branch March 26, 2025 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants