Geequlim / ECMAScript

Javascript binding for godotengine
MIT License
959 stars 84 forks source link

Refactor JavaScript integration and improve code readability. #164

Closed nmerget closed 11 months ago

nmerget commented 11 months ago

Hey, I'm not into C++ so I did some try/error to fix the issues for the CI.

This branch should be squashed 👍

fire commented 11 months ago

I approved the github actions. Been busy so if someone can continue the work on https://github.com/Geequlim/ECMAScript/pull/156 that'll be amazing.

fire commented 11 months ago

I think the js debugger is still being used so the build complains about needing it.

nmerget commented 11 months ago

@fire the pipeline is running now

The first run takes very long especially for windows, I don't know if this is a normal behavior when building the editor. After the cache is initialized it is way faster.

Please squash this branch before merging, I added a lot of testing commits because of the pipeline, we don't need them in the history

I propose merging the open PRs to have a working 4.1 branch. Afterward, I would like to structure the repo differently and add some extra workflow_calls to publish the artifacts: https://github.com/Geequlim/ECMAScript/issues/165

fire commented 11 months ago

Can you squash? I want to keep attribution intact

fire commented 11 months ago

Thanks