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

Fix glTF capitalization in class reference #95578

Merged

Conversation

tetrapod00
Copy link
Contributor

Fixes godotengine/godot-docs#9750 in the class reference.

Replaces user-facing mentions of "GLTF" with "glTF". This includes the class reference and the README.md of the glTF module.
Does not replace capitalized class names like GLTFAccessor. Does not affect any comments in .cpp or .h files.

There are some remaining user-facing uses of "GLTF" in error messages, for example here. I feel these are out of scope to fix in this PR.

ERR_PRINT("Error parsing GLTF light: The color must have exactly 3 numbers.");

@akien-mga akien-mga modified the milestones: 4.x, 4.4 Aug 16, 2024
@akien-mga akien-mga merged commit 4d3157c into godotengine:master Aug 16, 2024
18 checks passed
@akien-mga
Copy link
Member

Thanks!

@tetrapod00 tetrapod00 deleted the gltf-capitalization-classref branch August 16, 2024 16:07
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.

Capitalization of "GLTF" and "glTF" is not standardized across class reference and manual.
5 participants