-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
[3.6] Add C# code examples in Using signals
#7945
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for contributing to the .NET documentation!
Co-authored-by: Raul Santos <raulsntos@gmail.com>
Co-authored-by: Raul Santos <raulsntos@gmail.com>
4adbf63
to
cf315fb
Compare
Remove `EventHandler` suffix Co-authored-by: Raul Santos <raulsntos@gmail.com> Remove `EventHandler` suffix Remove `EventHandler` suffix Co-authored-by: Raul Santos <raulsntos@gmail.com> update
Match code examples Match code examples u Apply suggestions from code review Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
e085f8e
to
eb779b7
Compare
Using signals
Using signals
Co-authored-by: Raul Santos <raulsntos@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Using signals
Using signals
The changes can be cherry-picked into the 3.5/3.4 branches, the file is identical to 3.6, no need to open separate PR's. I will close the other 2 as superseded by this one, as this PR is reviewed. (For cherry-pick: Please cherry pick into 3.4 then removing the 3.5 label, we don't have a 3.4 label anymore) |
Thanks! Congrats on your first merged PR! I did a squash and merge for this PR but in the future please try and keep everything to 1 commit, if you don't know how to do that we have a guide here. |
Derived from this issue
Using signals
. #7917