Conversation
This comment was marked as outdated.
This comment was marked as outdated.
@marcinn we need to discuss that, details on Discord. |
This comment was marked as outdated.
This comment was marked as outdated.
09faeb7 to
0bee2d5
Compare
|
@carmel4a I'll split commits not related to GUT to a separate PR's. It is easier to maintain the project when PR's are as atomic as possible |
|
This branch requires #60 to be merge. |
b1f342c to
4f9afe4
Compare
2e6637b to
67d41ad
Compare
|
@JezSonic I've made some crucial changes to the code initially proposed by @carmel4a:
IMPORTANT NOTE: Before merging this PR a #60 should be merged first. I've built this branch top of ALSO please note that this PR includes some fixes of the |
| COPY SConstruct ./SConstruct | ||
| RUN scons | ||
|
|
||
| # Copy project files | ||
| COPY addons ./addons | ||
| COPY src ./src | ||
| RUN mkdir -p demo/addons demo/bin | ||
| RUN scons | ||
|
|
There was a problem hiding this comment.
Scons runned twice without need for that
There was a problem hiding this comment.
Must be run twice. Every docker build command creates a new layer (overrlayfs), which can be cached. First run is compiling godot-cpp. The second is compiling our code and linking them together, saving the result in a new layer.
There was a problem hiding this comment.
I think that you can use cache from scons builds for Godot cpp there tho, so docker wont need to build it at all if the cache will exist
6aec61e to
6d48880
Compare
6d48880 to
fe0b73a
Compare
Co-Authored-By: Marcin Nowak <marcin.j.nowak@gmail.com> Co-Authored-By: Kamil Lewan <kamil.lewan97@gmail.com> * Dockerfile for executing tests * GitHub workflow action * Example test * Added maszyna_debug=on|off to toggle libmaszyna internal debug macros * FIX: Mover's initialization routine in TrainController
fe0b73a to
e7c8ea4
Compare
|
Thanks! |
Adds GUT framework.
Requires #60 to be merged first.
Tests should inherit from
MaszynaGutTestdue to lack ofwait_idle_frames()bitwes/Gut#675