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

Set Godot 4.3 as minimum supported version #346

Merged
merged 5 commits into from
Dec 19, 2024
Merged

Conversation

wjt
Copy link
Member

@wjt wjt commented Dec 18, 2024

Godot 4.3 has been available for 4 months and it seems unlikely that anyone will have a pressing need to add blocks to a Godot 4.2 project.

Although we have been running unit tests against 4.2 in CI, those do not cover the full functionality of the plugin. To the best of my knowledge, no-one is regularly testing the plugin interactively against Godot 4.2.

Set the minimum version in the asset library template. Drop 4.2 from the CI test matrix. Remove some workarounds for 4.2 compatibility.

wjt added 4 commits December 18, 2024 11:29
This will make future diffs that add and remove engine versions more
legible.
Godot 4.3 has been available for 4 months and it seems unlikely that anyone
will have a pressing need to add blocks to a Godot 4.2 project.

Although we have been running unit tests against 4.2 in CI, those do not
cover the full functionality of the plugin. To the best of my knowledge,
no-one is regularly testing the plugin interactively against Godot 4.2.

Set the minimum version in the asset library template. Drop 4.2 from the
CI test matrix.
Now that Godot 4.3 is the minimum supported version, we can use this
new method to merge two dictionaries a little more tersely.
This backs out commit 99e2dc1.

We now require Godot 4.3.
@wjt wjt force-pushed the drop-godot-4.2-support branch from 809e24c to 2684d95 Compare December 18, 2024 11:58
@wjt wjt marked this pull request as ready for review December 18, 2024 11:59
Copy link
Contributor

@manuq manuq left a comment

Choose a reason for hiding this comment

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

Could you also address the "Polyfill of Node.is_part_of_edited_scene(), available to GDScript in Godot 4.3+." in addons/block_code/ui/util.gd ?

We now require Godot 4.3, which introduces this method.
@wjt
Copy link
Member Author

wjt commented Dec 18, 2024

Done

@wjt wjt requested a review from manuq December 18, 2024 18:44
@manuq manuq merged commit ba034b6 into main Dec 19, 2024
2 checks passed
@manuq manuq deleted the drop-godot-4.2-support branch December 19, 2024 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants