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

Remove 3 methods from GDExtension class, following upstream #611

Merged
merged 2 commits into from
Feb 19, 2024

Conversation

Bromeon
Copy link
Member

@Bromeon Bromeon commented Feb 19, 2024

The following methods could not be properly used; the GDExtension team thus decided to unbind them from the API despite a breaking change:

  • GDExtension::open_library
  • GDExtension::initialize_library
  • GDExtension::close_library

We follow suit by removing them from our API as well, thus preventing loading errors in the method tables.
This PR also changes both minimal and full CI to use full codegen, to make these issues more obvious.

Related Godot PR: godotengine/godot#88418

The following methods could not be properly used; it was thus decided to unbind them from the
GDExtension API despite a breaking change:

* GDExtension::open_library
* GDExtension::initialize_library
* GDExtension::close_library
@Bromeon Bromeon added quality-of-life No new functionality, but improves ergonomics/internals status: upstream Depending on upstream fix (typically Godot) c: engine Godot classes (nodes, resources, ...) labels Feb 19, 2024
@Bromeon Bromeon force-pushed the qol/adjust-removed-apis branch from e988812 to 0db2d36 Compare February 19, 2024 16:58
@GodotRust
Copy link

API docs are being generated and will be shortly available at: https://godot-rust.github.io/docs/gdext/pr-611

@Bromeon Bromeon enabled auto-merge February 19, 2024 17:03
@Bromeon Bromeon added this pull request to the merge queue Feb 19, 2024
Merged via the queue into master with commit 7848e28 Feb 19, 2024
16 checks passed
@Bromeon Bromeon deleted the qol/adjust-removed-apis branch February 19, 2024 17:13
@Bromeon Bromeon mentioned this pull request Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: engine Godot classes (nodes, resources, ...) quality-of-life No new functionality, but improves ergonomics/internals status: upstream Depending on upstream fix (typically Godot)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants