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

Add autocompletion for RenderingServer's global shader methods & has_os_feature #86798

Merged

Conversation

Mickeon
Copy link
Contributor

@Mickeon Mickeon commented Jan 4, 2024

Related to #86753, #86747, #86758, #86764, and #86777

This PR adds autocompletion to all of RenderingServer's methods that are about global shaders, allowing you to see the available ones right away, instead of having to check the Project Settings every time.

image

It also features... has_os_feature() because it's really nice to have a general clue as to what to put in there.

image

@Mickeon Mickeon requested a review from a team as a code owner January 4, 2024 19:31
@Mickeon Mickeon force-pushed the autocompletion-rendering-server branch from 130b7bc to d617111 Compare January 4, 2024 19:34
@YeldhamDev YeldhamDev added this to the 4.3 milestone Jan 4, 2024
@Mickeon Mickeon force-pushed the autocompletion-rendering-server branch from d617111 to 7eae72c Compare January 6, 2024 18:22
@Mickeon Mickeon force-pushed the autocompletion-rendering-server branch from 7eae72c to 472e3b3 Compare January 8, 2024 11:20
@Mickeon
Copy link
Contributor Author

Mickeon commented Jan 8, 2024

Updated this PR to cleanse @clayjohn's worries by surrounding the method with TOOLS_ENABLED.

Copy link
Member

@clayjohn clayjohn left a comment

Choose a reason for hiding this comment

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

Seems fine to me. I am not familiar with the editor-side of things, but from the rendering-side it looks fine

@akien-mga akien-mga merged commit a50a6b9 into godotengine:master Jan 9, 2024
15 checks passed
@akien-mga
Copy link
Member

Thanks!

@Mickeon Mickeon deleted the autocompletion-rendering-server branch January 9, 2024 10:35
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.

5 participants