-
-
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
Update the custom AudioStream tutorial to Godot 4 #9792
base: master
Are you sure you want to change the base?
Conversation
Are these changes specific to Godot 4.4 or are they relevant for 4.3 as well? |
I think they go all the way back to godot 4.0, since the tutorial was broken back then to. I'll try to recompile older versions of godot with the module as far back as I can to see where it starts breaking, and update you in a bit. |
Yup, it works on 4.0.4-stable too! |
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.
All code should be indented with four spaces instead of tabulators
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
contributing/development/core_and_modules/custom_audiostreams.rst
Outdated
Show resolved
Hide resolved
You will still need to fix the indentation and empty lines, otherwise the style looks good, will try to test this soon as well |
I think I fixed all the empty lines I missed before, and indentation. |
should fix #9231 #6170