Test CMake project with CI

[o01eg]

Includes #456 to fix compilation

[o01eg]

Ping!

[o01eg]

Set minimum python version requirement

[o01eg]

I've rebased PR and updated project for recent version of Godot.

[bradc6]

@Calinou Is this something that can be merged in?

[Calinou]

@Calinou Is this something that can be merged in?

I don't use CMake (and vnen doesn't either), so I don't know if the way this is implemented is correct.

Out of curiosity, why was a second CMakeLists.txt file was added in the test folder? Isn't the top-level CMakeLists.txt sufficient for this purpose?

[bradc6]

You want cmake lists files per sub section. Having the two is the standard way to do it

[bradc6]

@Calinou Would it be of any value for me to test it for validation to merge it in?

[Calinou]

@Calinou Would it be of any value for me to test it for validation to merge it in?

CI checks have passed, so I guess we can test this in production (especially since this won't be cherry-picked to a stable branch due to incompatibilities).

Feel free to open a pull request that targets the 3.x branch, so the CMake setup can also be tested there 🙂

[Calinou]

Thanks!

[o01eg]

Should I backport this PR to branchs for stable Godot version?

[Calinou]

Should I backport this PR to branchs for stable Godot version?

Yes, on 3.x specifically.