search

otros-systems / otroslogviewer

Log viewer focused on developers work
Apache License 2.0
424 stars 87 forks source link

Gradle update #604

Closed svennissel closed 2 years ago

svennissel commented 2 years ago

Thank you for your work @benlazaro. Now the build is state of the art. One difference to the older build are more dependencies and a bigger zip file. This is not nice but no problem. I think next step is to update dependencies and Java Version and cleanup not needed.

benlazaro commented 2 years ago

Great job fixing the conflicts also, @svennissel. We should create a list of tasks that need to be done for improving the project, so that we can start tackling them one by one and see how we’re progressing. Regarding the Uber-jar, imho I think it’s better to have all the dependency hidden from the end used to decrease the chances of accidental deletion or modification of the dependencies. This also help us simplify our code since we don’t need to have a lot of conditionals to make sure the dependencies were loaded. Another plus is that since now we have the dependencies embedded into the jar, loading should be a little faster too. Anyhow, this project is alive and a work in progress so we will continue improving it. :)