abstractfoundry / lumicube-daemon

9 stars 12 forks source link

Add basic GitHub action for running maven build and test #5

Closed mfoo closed 1 year ago

mfoo commented 1 year ago

This change adds a GitHub pipeline that runs a Maven build and JUnit tests to ensure that builds are green before code is merged. This should help prevent any contributions accidentally introducing issues.

Notes:

Having a build/test pipeline is a prerequisite to introducing something like Dependabot to automatically keep dependencies up-to-date - we don't want to accidentally merge dependency bumps that break the build :smile:

Edit:

Oops, forgot to say - you can see evidence that this works here:

abstractfoundry commented 1 year ago

Hi @mfoo thank you so much for putting this together, and for all your other great PRs and issues 😀 🏆 ✨ Looking forward to getting these all merged in! The reason we're still using JDK 16 is that we were getting segmentation faults in the arm32 C2 compiler (super bizarre, right) on more recent versions. Is it just a simple matter of changing java-version to '16' to continue building and testing against JDK 16, until we can be sure the compiler issue has been sorted out?

mfoo commented 1 year ago

Should be, yep!

abstractfoundry commented 1 year ago

Awesome, thanks - I'll make that change after merging.