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

Actualize code generation #3

Merged
merged 2 commits into from
Jul 27, 2023

Conversation

malec-palec
Copy link
Contributor

This is a tough one. I suggest not touching "legacy" implementation of ./src/EventsFunctionsExtensionsLoader/index.js (moved to ./src/EventsFunctionsExtensionsLoader/legacy.js in its original state) to work with GDevelop up to v5.1.149 inclusive.
So there will be no need to support a bunch of backward-compatibility if-else logic enlarging diff with GDevelop latest.

With these changes the Platformer game sample is exported without out-of-memory error and works as expected.

Meanwhile, the idea of

fully recreate this project in a way that extracts all this code from the GDevelop source code automatically.

is the right direction to go.

@arthuro555
Copy link
Owner

I'm mostly done with the rewrite :) I'll merge and publish this in the meantime, but don't put much effort in making other PRs since it'll all be replaced soon.

@arthuro555 arthuro555 merged commit 29d3b43 into arthuro555:master Jul 27, 2023
@malec-palec malec-palec deleted the actualize-code-generation branch July 27, 2023 22:36
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.

None yet

2 participants