-
-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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 documentation about .gdextension file #9383
Add documentation about .gdextension file #9383
Conversation
2b73f33
to
f567fa1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this looks great!
Only a few notes, mostly small wording changes for clarity.
b47f96d
to
90e7155
Compare
90e7155
to
ac065d9
Compare
ac065d9
to
91b93c8
Compare
91b93c8
to
cea6ef0
Compare
Thanks for the feedback! Should be good now |
cea6ef0
to
ecc21c6
Compare
Updated to feedback, added another sections about icons support and removed the icons section from the Example page as it makes no sense to have it double |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! This is looking very nice :-)
I just have a few suggestions, mostly small wording changes for clarity.
ecc21c6
to
0dda1da
Compare
Update tutorials/scripting/gdextension/gdextension_file.rst
0dda1da
to
7695474
Compare
@paddy-exe Great work! Merged :) Also thanks to reviewing everyone. Does this apply fully to 4.0/4.1/4.2? |
Thanks! :) This should apply for 4.0, 4.1 and 4.2 as well, yes. |
Cherry-picked to 4.2 in #9648. |
This is a PR in the effort to improve the GDExtension docs for users
Adds details how to configure the
.gdextension
file and aids as a quick reference guide to add support for platforms