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

Add get_script_instance to GDExtension #80040

Merged

Conversation

fuzzybinary
Copy link
Contributor

@fuzzybinary fuzzybinary commented Jul 30, 2023

Adds a method to GDExtension to get the ScriptInstanceExtension for an object.

This implements the proposal in godotengine/godot-proposals#7199

@AThousandShips AThousandShips added this to the 4.x milestone Jul 30, 2023
@fuzzybinary
Copy link
Contributor Author

Comments fixed and clang-format run. Sorry, I'm very used to it running automatically and forgot to set it up.

fuzzybinary added a commit to fuzzybinary/godot_dart that referenced this pull request Jul 30, 2023
Casting to script types requires [this PR](godotengine/godot#80040), which will make this extension require Godot 4.2 when it is merged.
Copy link
Contributor

@dsnopek dsnopek left a comment

Choose a reason for hiding this comment

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

This looks great to me!

However, it could use a rebase to force the tests to re-run: the test failure is left over from when we were running out of disk space.

@fuzzybinary fuzzybinary force-pushed the add-object-get-script-instance branch from 27592bc to 9ee4825 Compare August 7, 2023 21:57
@fuzzybinary
Copy link
Contributor Author

Rebased and pushed!

@dsnopek
Copy link
Contributor

dsnopek commented Aug 8, 2023

Thanks!

Unfortunately, you're also going to need to squash this into a single commit per Godot's Git workflow:

https://docs.godotengine.org/en/stable/contributing/workflow/pr_workflow.html#the-interactive-rebase

Sorry I didn't catch that when I was reviewing earlier

Adds a method to GDExtension to get the ScriptInstanceExtension for an object.
@fuzzybinary fuzzybinary force-pushed the add-object-get-script-instance branch from 9ee4825 to efc8fd6 Compare August 8, 2023 03:09
@fuzzybinary
Copy link
Contributor Author

Fixed 😅

@akien-mga akien-mga changed the title Add get_script_instance to GDExtension Add get_script_instance to GDExtension Aug 8, 2023
@akien-mga akien-mga modified the milestones: 4.x, 4.2 Aug 8, 2023
@akien-mga akien-mga merged commit 5bd5698 into godotengine:master Aug 8, 2023
@akien-mga
Copy link
Member

Thanks!

@fuzzybinary fuzzybinary deleted the add-object-get-script-instance branch August 8, 2023 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add get_script_instance() to GDExtension
5 participants