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

Rename shading language to GDShader #9984

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

tetrapod00
Copy link
Contributor

@tetrapod00 tetrapod00 commented Sep 21, 2024

Closes godotengine/godot-proposals#4350.
Closes godotengine/godot-proposals#10784.

Companion PR to godotengine/godot#97298.

This is a placeholder draft. If godotengine/godot-proposals#10784 is approved, I'll go through the manual and make the required changes.

The places that definitely need to be changed are Shading Language/Introduction, the Shader class reference, and the GLSL Migration Guide. In places like compute shaders where godot does use GLSL, we should make the distinction that "GLSL is used, not GDShader"

Unlike the class ref, for the manual we don't need to replace every reference to the shading language with "GDShader" immediately, to avoid breaking translations.

@Calinou Calinou added enhancement area:manual Issues and PRs related to the Manual/Tutorials section of the documentation area:contributing Issues and PRs related to the Contributing/Development section of the documentation cherrypick:4.3 labels Sep 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:contributing Issues and PRs related to the Contributing/Development section of the documentation area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.3 enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Formally rename the Godot shading language to "GDShader" Give a consistent name to Godot's shading language
2 participants