Skip to content

Conversation

StephaneDelcroix
Copy link
Contributor

Summary

Fix the sample for EventHandler so the handler signature matches the delegate, and avoids warning

@StephaneDelcroix StephaneDelcroix requested a review from a team as a code owner December 9, 2024 09:23
@ghost ghost added the area-Infrastructure label Dec 9, 2024
@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Dec 9, 2024

This comment was marked as outdated.

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

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

Thanks @StephaneDelcroix. I updated the others in the directory to match too.

@gewarren gewarren enabled auto-merge (squash) December 9, 2024 18:43
Copy link

Learn Build status updates of commit 08ddb0b:

✅ Validation status: passed

File Status Preview URL Details
snippets/csharp/System/EventArgs/Overview/program.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/programnodata.cs ✅Succeeded View
snippets/csharp/System/EventArgs/Overview/programtruncated.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/programwithdata.cs ✅Succeeded View
snippets/csharp/System/EventArgs/Overview/programwithdelegate.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/Project.csproj ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

Copy link

Learn Build status updates of commit 354839b:

✅ Validation status: passed

File Status Preview URL Details
snippets/csharp/System/EventArgs/Overview/program.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/programnodata.cs ✅Succeeded View
snippets/csharp/System/EventArgs/Overview/programtruncated.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/programwithdata.cs ✅Succeeded View
snippets/csharp/System/EventArgs/Overview/programwithdelegate.cs ✅Succeeded
snippets/csharp/System/EventArgs/Overview/Project.csproj ✅Succeeded

For more details, please refer to the build report.

For any questions, please:

@gewarren gewarren merged commit bc434e6 into dotnet:main Dec 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Infrastructure community-contribution Indicates that the PR has been added by a community member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants