Skip to content

Conversation

whtsht
Copy link

@whtsht whtsht commented Sep 18, 2023

@raulsntos raulsntos added enhancement topic:dotnet area:getting started Issues and PRs related to the Getting Started section of the documentation labels Sep 18, 2023
@raulsntos raulsntos added this to the 3.x milestone Sep 18, 2023
Copy link
Member

@raulsntos raulsntos left a 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!

@raulsntos raulsntos linked an issue Sep 18, 2023 that may be closed by this pull request
whtsht and others added 2 commits September 21, 2023 09:45
Co-authored-by: Raul Santos <raulsntos@gmail.com>
Co-authored-by: Raul Santos <raulsntos@gmail.com>
@whtsht whtsht force-pushed the 3.6-update-getting-started branch 3 times, most recently from 4adbf63 to cf315fb Compare September 21, 2023 01:35
whtsht and others added 2 commits September 21, 2023 10:36
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>
@whtsht whtsht force-pushed the 3.6-update-getting-started branch 2 times, most recently from e085f8e to eb779b7 Compare September 21, 2023 01:39
@whtsht whtsht requested a review from raulsntos September 21, 2023 05:21
@AThousandShips AThousandShips changed the title Add C# code examples in Using signals [3.x] Add C# code examples in Using signals Sep 21, 2023
Co-authored-by: Raul Santos <raulsntos@gmail.com>
Copy link
Member

@raulsntos raulsntos left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@whtsht whtsht changed the title [3.x] Add C# code examples in Using signals [3.6] Add C# code examples in Using signals Sep 27, 2023
@Piralein
Copy link
Member

Piralein commented Sep 27, 2023

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)

@skyace65 skyace65 merged commit 5d6433d into godotengine:3.6 Nov 24, 2023
@skyace65
Copy link
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:getting started Issues and PRs related to the Getting Started section of the documentation cherrypick:3.5 enhancement topic:dotnet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[3.x] There are no C# code examples in Using signals.
5 participants