Module tweaks to reduce warnings

Author: prdoyle

bosk-core/src/main/java/module-info.java

@@ -1,10 +1,10 @@
 module works.bosk.core {
-	requires org.jetbrains.annotations;
+	requires transitive org.jetbrains.annotations;
 	requires org.objectweb.asm;
 	requires org.objectweb.asm.util;
 	requires org.pcollections;
 	requires org.slf4j;
-	requires works.bosk.annotations;
+	requires transitive works.bosk.annotations;
 
 	requires static lombok;
 

bosk-jackson/src/main/java/module-info.java

@@ -1,8 +1,8 @@
 module works.bosk.jackson {
-	requires com.fasterxml.jackson.core;
-	requires com.fasterxml.jackson.databind;
+	requires transitive com.fasterxml.jackson.core;
+	requires transitive com.fasterxml.jackson.databind;
 	requires org.slf4j;
-	requires works.bosk.core;
+	requires transitive works.bosk.core;
 
 	requires static lombok;
 

bosk-logback/src/main/java/module-info.java

@@ -1,7 +1,8 @@
 module works.bosk.logback {
-	requires ch.qos.logback.classic;
-	requires ch.qos.logback.core;
-	requires org.slf4j;
-	requires works.bosk.core;
+	requires transitive ch.qos.logback.classic;
+	requires transitive ch.qos.logback.core;
+	requires transitive org.slf4j;
+	requires transitive works.bosk.core;
+
 	exports works.bosk.logback;
 }

bosk-mongo/src/main/java/module-info.java

@@ -3,7 +3,7 @@
 	requires org.mongodb.bson;
 	requires org.mongodb.driver.core;
 	requires org.slf4j;
-	requires works.bosk.core;
+	requires transitive works.bosk.core;
 
 	requires static lombok;
 

bosk-opentelemetry/src/main/java/module-info.java

@@ -1,7 +1,7 @@
 module works.bosk.opentelemetry {
 	requires io.opentelemetry.api;
 	requires io.opentelemetry.context;
-	requires works.bosk.core;
+	requires transitive works.bosk.core;
 
 	exports works.bosk.opentelemetry;
 }

bosk-spring-boot/src/main/java/module-info.java

@@ -6,7 +6,7 @@
 	requires spring.boot;
 	requires spring.context;
 	requires spring.web;
-	requires works.bosk.core;
+	requires transitive works.bosk.core;
 	requires works.bosk.jackson;
 
 	requires static lombok;

bosk-sql/src/main/java/module-info.java

@@ -1,8 +1,8 @@
 module works.bosk.sql {
-	requires com.fasterxml.jackson.databind;
-	requires org.jooq;
+	requires transitive com.fasterxml.jackson.databind; // This really shouldn't be transitive
+	requires transitive org.jooq;
 	requires org.slf4j;
-	requires works.bosk.core;
+	requires transitive works.bosk.core;
 	requires works.bosk.jackson;
 
 	exports works.bosk.drivers.sql;

build.gradle

@@ -56,6 +56,8 @@ configure(subprojects.findAll {
 		options.compilerArgs << '-Xlint:-serial'     // Don't care about Java serialization
 		options.compilerArgs << '-Xlint:-try'        // Really annoying bogus "auto-closeable never used" warnings
 		options.compilerArgs << '-Xlint:-processing' // Not all annotations are meant for compile-time "annotation processing"
+		options.compilerArgs << '-Xlint:-missing-explicit-ctor' // We probably ought to fix this instead of suppressing it
+		options.compilerArgs << '-Xlint:-module' // We probably ought to fix this instead of suppressing it
 		options.forkOptions.jvmArgs << '--sun-misc-unsafe-memory-access=allow' // For Lombok
 	}