Skip to content

Fix vararg methods forwarded to the ClassDB singleton#1485

Merged
dsnopek merged 1 commit into
godotengine:masterfrom
dsnopek:classdb-call-static-method
Jun 14, 2024
Merged

Fix vararg methods forwarded to the ClassDB singleton#1485
dsnopek merged 1 commit into
godotengine:masterfrom
dsnopek:classdb-call-static-method

Conversation

@dsnopek

@dsnopek dsnopek commented Jun 12, 2024

Copy link
Copy Markdown
Collaborator

Aims to fix the CI issues with Godot PR godotengine/godot#93141 as well as allowing forwarded vararg methods to work.

This will need to be cherry-picked to the 4.2 branch before CI on that PR can pass.

@dsnopek dsnopek added bug This has been identified as a bug enhancement This is an enhancement on the current functionality cherrypick:4.1 cherrypick:4.2 labels Jun 12, 2024
@dsnopek dsnopek added this to the 4.x milestone Jun 12, 2024
@dsnopek
dsnopek requested a review from a team as a code owner June 12, 2024 13:22
@dsnopek

dsnopek commented Jun 12, 2024

Copy link
Copy Markdown
Collaborator Author

PR #1486 should fix the test failure - it's unrelated to the changes here

@dsnopek
dsnopek force-pushed the classdb-call-static-method branch from 045852d to e04a26b Compare June 12, 2024 15:32
@AThousandShips

AThousandShips commented Jun 14, 2024

Copy link
Copy Markdown
Member

My bad accidentally made my own fix for this, will make that just an improvement and leave this for the immediate fix

@dsnopek

dsnopek commented Jun 14, 2024

Copy link
Copy Markdown
Collaborator Author

Thanks for the review!

@dsnopek
dsnopek merged commit 64f1bc8 into godotengine:master Jun 14, 2024
@dsnopek

dsnopek commented Jun 14, 2024

Copy link
Copy Markdown
Collaborator Author

Cherry-picked for 4.1 in PR #1491

@dsnopek

dsnopek commented Jun 14, 2024

Copy link
Copy Markdown
Collaborator Author

Cherry-picked for 4.2 in PR #1492

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug This has been identified as a bug enhancement This is an enhancement on the current functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants