syd711 / vpin-studio

Competition, table, and player management for VPins.
MIT License
32 stars 2 forks source link

Missing github ci build #8

Closed francisdb closed 1 year ago

francisdb commented 1 year ago

mvn clean test fails here

Would be great if everything at least builds on win / mac / linux and has a matrix github ci build for every pr / commit to main.

This also means no dependencies on existing file structure outside the project dir

syd711 commented 1 year ago

Yes, I'm aware of this pile of shame. Right now the test are more or less some tooling utils but far away from production. I can't say when or if this ever will reach a "production" state as this is a one-man show and I'm usually more focussed on new features than technical debts (except bugs).

francisdb commented 1 year ago

I might be able to help but but let me first try to talk to you on discord

francisdb commented 1 year ago

Not a pile of shame, great tool. Just not ready for other ppl to get involved.

syd711 commented 1 year ago

There is a embedded testsystem available now. I haven't deleted my local environment to check if there are still hidden references, but I think I covered everything.

francisdb commented 1 year ago

still not compiling on mac here:

UnsatisfiedLinkError: 'java.lang.Integer org.jnativehook.GlobalScreen.getAutoRepeatRate()'

What is that used for?

syd711 commented 1 year ago

Looks like it belongs to JavaFX? I tried stackoverflow, but since I can't reproduce it without a mac its tricky for me.

francisdb commented 1 year ago

Strange that this fails on the server module

image
francisdb commented 1 year ago

anyway, don't have the time to look into this now