Skip to content
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 C# signal documentation #6110

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

raulsntos
Copy link
Member

@raulsntos raulsntos commented Aug 26, 2022

Updates C# signal documentation and code examples to the new API in 4.0.

Copy link
Member

@mhilbrunner mhilbrunner left a comment

Choose a reason for hiding this comment

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

The changes to the classes/*.rst files need to be made to the class doc XMLs in the main repo instead and removed from these, as the RSTs are autogenerated and overwritten periodically, so these changes would get lost.

Everything else looks good at first glance.

@raulsntos
Copy link
Member Author

raulsntos commented Aug 26, 2022

Oops, you are right, I used a regex to find all instances and missed that I was editing the class reference 🙃

Edit: Extracted to godotengine/godot#64930

Updates C# signal documentation and code examples to the new API in 4.0
@mhilbrunner mhilbrunner merged commit c2f7474 into godotengine:master Aug 26, 2022
@mhilbrunner
Copy link
Member

Great, thanks for working on 4.0 docs! ❤️

@raulsntos raulsntos deleted the dotnet/new-signals branch August 26, 2022 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants