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

Documentation for shader built-in include files (godotengine/godot#94193) #10703

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Jackerty
Copy link
Contributor

Documents merged feature godotengine/godot#94193 to the end of the compute shader tutorial.

If someone has a better example in mind I'm open to suggestions. I just wanted something short that does not have too much extra code around the include.

)

Documents merged feature godotengine/godot#94193 to the end of the compute shader tutorial.

If someone has a better example in mind I'm open to suggestions. I just wanted something short that isn't too much extra code around the include.
@skyace65 skyace65 added area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:shaders labels Feb 22, 2025
@skyace65 skyace65 added this to the 4.4 milestone Feb 22, 2025
@skyace65 skyace65 requested review from BastiaanOlij and tetrapod00 and removed request for BastiaanOlij and tetrapod00 February 22, 2025 12:56
-----------------
.. warning::

The feature is still been baked!
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
The feature is still been baked!
The feature is experimental.

The feature is still been baked!

I'm not sure how this is noted elsewhere, but it seems better to use the wording experimental rather than the concept of feature baking. It's unclear how a software feature is related to cooking.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Do think that warning is excessive?

Considering what DastiaanOlij said in his PR I'm a bit torn about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:manual Issues and PRs related to the Manual/Tutorials section of the documentation topic:shaders
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants