Conversation
|
I'm not sure if it makes sense to merge this into |
|
I think it makes sense to add this as a bit of ease for understanding. When you submit PRs, you can then very quickly see if the code or the tests are not working and quickly debug. The normal solution is to open the failed builds and read the errors which is valid but an alternative others do not always prefer. I don't think it adds any bloat, given we only have 3 workflows total. It adds extra functionality - if very little - to lib199 GitHub, but does not pose any significant problems in return. |
|
Both of those error conditions should be visible in the ide. Imo, separating the build and test phases only makes sense when working with a more complex test-set than what we have here. That being said, if you think this is valuable, I agree that the downsides aren't too excessive. At the very least, however, I think it might be better to merge all of these into the Java CI file. Either as two jobs or by splitting the build step into a separate build and test step. |
build but without tests