-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Update getting started C# code, remove randomize #6811
Merged
mhilbrunner
merged 2 commits into
godotengine:master
from
raulsntos:dotnet/getting_started
Mar 9, 2023
Merged
Update getting started C# code, remove randomize #6811
mhilbrunner
merged 2 commits into
godotengine:master
from
raulsntos:dotnet/getting_started
Mar 9, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
raulsntos
added
bug
enhancement
topic:dotnet
area:manual
Issues and PRs related to the Manual/Tutorials section of the documentation
area:getting started
Issues and PRs related to the Getting Started section of the documentation
and removed
area:manual
Issues and PRs related to the Manual/Tutorials section of the documentation
labels
Feb 24, 2023
raulsntos
force-pushed
the
dotnet/getting_started
branch
from
February 24, 2023 04:24
11c3ce9
to
1ff6147
Compare
RedworkDE
reviewed
Feb 24, 2023
raulsntos
force-pushed
the
dotnet/getting_started
branch
from
March 3, 2023 14:47
8253dd8
to
3d7d26e
Compare
- Sync C# code with GDScript. - Remove `randomize` calls. - Rename members that have been renamed in Godot's C# API for 4.0. - Change `delta` parameter type to `double` in C#. - Follow our C# code style more closely. - Other minor code fixes.
raulsntos
force-pushed
the
dotnet/getting_started
branch
from
March 6, 2023 16:24
3d7d26e
to
7a13309
Compare
Co-authored-by: RedworkDE <[email protected]>
raulsntos
force-pushed
the
dotnet/getting_started
branch
from
March 6, 2023 16:27
7a13309
to
63701bd
Compare
mhilbrunner
approved these changes
Mar 9, 2023
Looks good, merged! Thanks. 🎉 |
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
bug
enhancement
topic:dotnet
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
randomize
calls.delta
parameter type todouble
in C#.