-
-
Notifications
You must be signed in to change notification settings - Fork 22k
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
doc: Use Signal.emit
instead of emit_signal
in Object examples
#69662
Conversation
This PR tries changes the function shown in the examples of |
Well, we're using Signal's API for connecting and disconnecting, for example. So I think this change is valid.
You are probably right there. |
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.
I think this change is good by itself, regardless of possible further improvements of the whole section. Moving away from strings in favor of signal names wherever we can in examples is good, IMO.
Unfortunately, this needs a rebase.
Signal.emit
instead of emit_signal
in Object examples
Rebased, and gave the PR and commit a meaningful title. Would be good that @godotengine/dotnet checks that the C# example is correct. |
Hi. Nope, this does not work in C#. Currently, the only way to emit signals is through |
@paulloz I restored the C# code to its original state. Sorry, I’m not familiar with C#. |
Looks good! Could you squash the commits? See PR workflow for instructions. Edit: Done myself. |
@akien-mga I am not a professional programmer. I have been reading PR workflow and learning git, and it may take a few days. If you have already helped me complete the "rebase", I am very grateful.😊 |
Yes it's all good now :) I'll merge this soon. |
Thanks! And congrats for your first merged Godot contribution 🎉 And apologies for the delay getting this reviewed. |
Change the code and use the new method of godot4
URL:https://docs.godotengine.org/en/latest/classes/class_object.html