镜像站点 · 本页由第三方 GitHub 只读镜像提供,非 GitHub 官方站点,不接受任何登录或凭据输入。前往 github.com
Skip to content

Document SYSLIB1219 options validation diagnostic - #56288

Merged
gewarren merged 2 commits into
dotnet:mainfrom
adarsh6980:patch-1
Oct 5, 2026
Merged

gewarren merged 2 commits into
dotnet:mainfrom
adarsh6980:patch-1

Conversation

@adarsh6980

@adarsh6980 adarsh6980 commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #56285.

Adds the missing SYSLIB1219 row to the options-validation source-generator diagnostics table. The wording follows the existing SYSLIB1205 row for Validate, using ValidateAsync for its async counterpart.

Checked the diagnostic descriptor in dotnet/runtime's Microsoft.Extensions.Options generator, which defines SYSLIB1219 as AlreadyImplementsValidateAsyncMethod. The SYSLIB1218 keyword is left unchanged because its intended use has not been confirmed.

Validation: inspected the rendered Markdown and verified that the diff is one table-row addition. No product code changed.


Internal previews

File Preview link
docs/fundamentals/syslib-diagnostics/syslib1201-1219.md Learn preview

Build report

Copilot AI balanced review requested due to automatic review settings September 30, 2026 08:41
@adarsh6980
adarsh6980 requested a review from a team as a code owner September 30, 2026 08:41
@dotnetrepoman dotnetrepoman Bot added this to the September 2026 milestone Sep 30, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@dotnet-policy-service dotnet-policy-service Bot added dotnet-fundamentals/svc community-contribution Indicates PR is created by someone from the .NET community. labels Sep 30, 2026
Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
@gewarren
gewarren enabled auto-merge (squash) October 2, 2026 16:15
@gewarren
gewarren requested a review from BillWagner October 2, 2026 16:16

@BillWagner BillWagner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This LGTM as well.

@gewarren
gewarren merged commit 7cd24df into dotnet:main Oct 5, 2026
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-fundamentals/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SYSLIB1219 (and placeholder SYSLIB1218) missing from the options-validation source-generator diagnostics reference

4 participants