michalakadam
commented
5 years ago Cieszę się, że nie klepnąłeś tak o tej recki, tylko chwilę nad nią pomyślałeś! Rzeczywiście server/pom.xml w ogóle nie dotykałem i porządek jest tam wskazany. Jeśli chodzi o wersję rodzica, nie znalazłem dobrego rozwiązania. Podmoduły muszą mieć określoną wersję rodzica, bez niej projekt się nie buduje. https://stackoverflow.com/questions/10582054/maven-project-version-inheritance-do-i-have-to-specify-the-parent-version Jeśli masz na to jakiś pomysł, daj znać!
Jest jeszcze jedna rzecz, nad którą się zastanawiam. FindBugs wyrzuca ostrzeżenia przy budowaniu, które nie wpływają na sposób działania wtyczki ani Mavena, ale nie wyglądają elegancko. https://github.com/gradle/gradle/issues/6725 Są dwa rozwiązania: olewamy to, bo to błąd w gradlu, a możliwe rozwiązania nie są polecane i mogą wpłynąć na działanie programu: https://stackoverflow.com/questions/46454995/how-to-hide-warning-illegal-reflective-access-in-java-9-without-jvm-argument albo jedziemy ze spotBugs. Jestem w stanie się przygotować do tej drugiej opcji. Poprawki wrzucone, czekam na reckę :)
Blysku
Not necessary plugins and dependencies removed. Versions of all plugins and dependencies hardcoded. FindBugs and pmd plugins switched on - warnings thrown by pmd fixed. Classes Listener and LoggedClient changed - watch out for merge conflicts!