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

Support Godot 3.4 #829

Merged
merged 2 commits into from
Dec 15, 2021
Merged

Support Godot 3.4 #829

merged 2 commits into from
Dec 15, 2021

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Dec 13, 2021

Updates api.json, CI scripts and examples.

This is a breaking change. Some GDNative APIs added default parameters, which can currently not be modeled in a backwards-compatible way in godot-rust (Rust itself doesn't support default parameters). It is however possible to manually generate api.json for an older Godot version (3.2 or 3.3) and keep using godot-rust normally.

Addresses #814 in the short-term 🛠️
Still on the radar: #640 📡

@Bromeon Bromeon added feature Adds functionality to the library c: bindings Component: GDNative bindings (mod api) breaking-change Issues and PRs that are breaking to fix/merge. labels Dec 13, 2021
@Bromeon Bromeon added this to the v0.10.0 milestone Dec 13, 2021
@Bromeon
Copy link
Member Author

Bromeon commented Dec 15, 2021

Merging, so other PRs can be tested against Godot 3.4.

bors r+

bors bot added a commit that referenced this pull request Dec 15, 2021
829: Support Godot 3.4 r=Bromeon a=Bromeon

Updates api.json, CI scripts and examples.

This is a **breaking change**. Some GDNative APIs added default parameters, which can currently not be modeled in a backwards-compatible way in godot-rust (Rust itself doesn't support default parameters). It is however possible to manually generate `api.json` for an older Godot version (3.2 or 3.3) and keep using godot-rust normally.

Addresses #814 in the short-term 🛠️ 
Still on the radar: #640 📡 

Co-authored-by: Jan Haller <[email protected]>
@bors
Copy link
Contributor

bors bot commented Dec 15, 2021

Canceled.

@Bromeon
Copy link
Member Author

Bromeon commented Dec 15, 2021

bors r+

@bors
Copy link
Contributor

bors bot commented Dec 15, 2021

Build succeeded:

@bors bors bot merged commit d9a9f9b into godot-rust:master Dec 15, 2021
@Bromeon Bromeon deleted the feature/godot-3.4 branch December 15, 2021 18:15
@Bromeon Bromeon mentioned this pull request Dec 23, 2021
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change Issues and PRs that are breaking to fix/merge. c: bindings Component: GDNative bindings (mod api) feature Adds functionality to the library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant