Merge commit godotengine/godot-cpp@08fd033 #29
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… for generated function call bindings
CMake: Pre-Commit Hook to Gersemi for auto-formatting
People seem to want to build godot-cpp without any arguments, not even specifying a target. We have to detect if we are top level though because it conflicts with being consumed.
CMake: template_debug as default target when godot-cpp is top level
CMake: Comment Cleanup
The SCons build doesn't do this, so neither should CMake Excpetions are already handled by DISABLE_EXCEPTIONS option
Add GODOTCPP_TARGET configuration option Remove loop to generate the godot-cpp.<target> CMake Targets Rename test bindings target Update documentation
Remove unnecessary forced -fno-exceptions for web builds
CMake: Revert to single cmake target
Check that precision of `extension_api.json` matches build options
Replace header guards style with `#pragma once`
Fix -Wmissing-braces warning (included in -Wall) in array constructor for generated function call bindings
When attempting to generate XCode projects it would fail due to the target dependency chain not meeting expectations. This PR, adds the required dependency infomation so that the XCode generator works.
Made all warnings private. Warning as error private Consistency in generator expressions
It should be ok when folders exist. Exception shouldn't be thrown. Update binding_generator.py It should be ok when folds exist. It will fail to build without this patch, in vs code on windows with compiler ( visual studio community 2022 amd 64) . Co-Authored-By: Chris Cranford <ccranfor@redhat.com>
CMake: XCode dependency chain fixes
binding_generator.py: Don't error if directory already exists
CMake: public/private flags
Synchronize most shared variant code with Godot 4.4
Synchronize most shared template code with Godot 4.4
- Duplicated the ci.yml into ci-scons.yml and ci-cmake.yml - Replaced some manual shell commands to detect changes with tj-actions/changed-files@v45 - Conditionally run the CI depending on what files change. - Add ios toolchain from leetal/ios-cmake - Added a cmake-minimum of 3.10 to silence errors from ios toolchain
Update CI with new CMake builds.
Update test project to Godot 4.4
…-support Typed dictionary initializer support
Add support for `initializer_list` to Array and TypedArray
Fix compatibility break for CI action 'setup-godot-cpp'
Use Math::abs to avoid ambiguity with integer abs
Fix TypedDictionary binding generation
CI: bump of mozilla-sccache action to 0.0.9
Use Math::abs to avoid ambiguity with integer abs
SCons: Add `CPPEXTPATH` for external includes
CMake: Match #1758 move of DEBUG_METHODS_ENABLED from build options to defs.hpp
Register the classes used with the Godot editor
Spartan322
force-pushed
the
merge/08fd033
branch
2 times, most recently
from
728023b to
2c60395
Compare
Spartan322
force-pushed
the
merge/08fd033
branch
from
2c60395 to
24c03d8
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
14 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.