Skip to content

Modernize snippets (System.Diagnostics.Eventing.Reader) - #13040

Open
gewarren wants to merge 2 commits into
dotnet:mainfrom
gewarren:modernize-system-diagnostics-eventing-reader
Open

Modernize snippets (System.Diagnostics.Eventing.Reader)#13040
gewarren wants to merge 2 commits into
dotnet:mainfrom
gewarren:modernize-system-diagnostics-eventing-reader

Conversation

@gewarren

@gewarren gewarren commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Copilot AI lite review requested due to automatic review settings August 27, 2026 21:10
@gewarren
gewarren requested a review from a team as a code owner August 27, 2026 21:10

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.

Pull request overview

Updates the C# snippet for System.Diagnostics.Eventing.Reader.EventLogInformation.Attributes as part of the broader effort in #12713 to modernize snippet projects for .NET 10 and current C# patterns.

Changes:

  • Added an SDK-style snippet project file for the EventLogInformation/Attributes snippet directory.
  • Modernized the snippet code to use a using declaration for EventLogSession and moved snippet markers to focus on the demonstrated API.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
snippets/csharp/System.Diagnostics.Eventing.Reader/EventLogInformation/Attributes/Project.csproj Adds a compilable snippet project targeting .NET 10 with required package reference.
snippets/csharp/System.Diagnostics.Eventing.Reader/EventLogInformation/Attributes/attributes.cs Modernizes setup code and narrows the snippet region to the Attributes property usage.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@gewarren
gewarren enabled auto-merge (squash) August 27, 2026 22:30
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.

2 participants