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

Update class_raycast3d.rst per Issue #100139 #10485

Closed
wants to merge 1 commit into from

Conversation

jburt137
Copy link

@jburt137 jburt137 commented Jan 8, 2025

Found this issue on "up-for-grabs". I been wanting to contribute to more open source projects and Godot is a very exciting project I'd like to help out with based upon my background in VFX and animation.

Please review my updates and let me know what should be changed, how I can improve the doc updates since I'm new at this. Thank you!

godotengine/godot#100139

@jburt137
Copy link
Author

jburt137 commented Jan 8, 2025

Thank you!

@mhilbrunner
Copy link
Member

Hey, thanks for contributing! We can certainly use more help :)

Note that at the top of the file, it says:

.. DO NOT EDIT THIS FILE!!!
.. Generated automatically from Godot engine sources.
.. Generator: https://github.com/godotengine/godot/tree/master/doc/tools/make_rst.py.
.. XML source: https://github.com/godotengine/godot/tree/master/doc/classes/RayCast3D.xml.

The class reference (everything under classes/) is auto-generated from the XML files in the main engine repository, so you need to make any changes to the file there: https://github.com/godotengine/godot/tree/master/doc/classes/RayCast3D.xml

Please redo the changes in that file and submit a new PR in the engine repository. :)

@jburt137
Copy link
Author

jburt137 commented Jan 8, 2025

Ah, thank you for the clarification. I was wondering about that. I didn't know if it meant just don't edit the header or what, figured I'd learn as I did the process. I will edit the xml file instead. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants