Skip to content

Conversation

@hugarty
Copy link
Contributor

@hugarty hugarty commented Nov 7, 2023

Based in godotengine/godot#84456

Change register_tts_types() for initialize_tts_module()
Change unregister_tts_types() for uninitialize_tts_module()

@hugarty
Copy link
Contributor Author

hugarty commented Nov 7, 2023

Related to these comment

What should I do ?

@AThousandShips
Copy link
Member

Please restore the spaces, it should be spaces and not tabs

@AThousandShips
Copy link
Member

What should I do ?

Leave it for a separate decision, let's keep this focused

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

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

LGTM

Can you please squash your commits into one, see here

@AThousandShips AThousandShips added area:engine details Issues and PRs related to the Engine Details section of the documentation content:proofreading Issues and PRs related to proofreading the documentation cherrypick:4.1 cherrypick:4.0 labels Nov 7, 2023
…_libraries.rst

Based in godotengine/godot#84456

Change register_tts_types() to initialize_tts_module()
Change unregister_tts_types() to uninitialize_tts_module()
Add parameter ModuleInitializationLevel p_level to uninitialize_tts_module and initialize_tts_module
@mhilbrunner mhilbrunner merged commit 1145e1a into godotengine:master Nov 9, 2023
@mhilbrunner
Copy link
Member

mhilbrunner commented Nov 9, 2023

Merged. Thanks and congrats on your first merged contribution! PR welcome to update the writing custom unit tests page too :)

mhilbrunner added a commit to mhilbrunner/godot-docs that referenced this pull request Nov 11, 2023
Update register_types in binding_to_external_libraries.rst
@mhilbrunner
Copy link
Member

Cherry-picked to 4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:engine details Issues and PRs related to the Engine Details section of the documentation cherrypick:4.0 content:proofreading Issues and PRs related to proofreading the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants