This is an automated email from the ASF dual-hosted git repository.

dimas pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/polaris.git


The following commit(s) were added to refs/heads/main by this push:
     new a068689d Move dummy package-info.java to Readme.java (#916)
a068689d is described below

commit a068689d869c0380e7df589dbe5e612f4a3470c8
Author: Dmitri Bourlatchkov <[email protected]>
AuthorDate: Mon Feb 3 12:28:19 2025 -0500

    Move dummy package-info.java to Readme.java (#916)
    
    This is to avoid spurious error messages during build, such as:
    `error: No public or protected classes found to document`
---
 quarkus/defaults/build.gradle.kts                            |  2 --
 .../service/defaults/{package-info.java => Readme.java}      | 12 ++++++++----
 quarkus/server/build.gradle.kts                              |  2 --
 .../server/quarkus/{package-info.java => Readme.java}        | 11 ++++++++---
 4 files changed, 16 insertions(+), 11 deletions(-)

diff --git a/quarkus/defaults/build.gradle.kts 
b/quarkus/defaults/build.gradle.kts
index 73e86405..d3f33d9c 100644
--- a/quarkus/defaults/build.gradle.kts
+++ b/quarkus/defaults/build.gradle.kts
@@ -38,5 +38,3 @@ dependencies {
   compileOnly("io.quarkus:quarkus-opentelemetry")
   compileOnly("io.quarkus:quarkus-smallrye-context-propagation")
 }
-
-tasks.withType<Javadoc> { isFailOnError = false }
diff --git 
a/quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/package-info.java
 
b/quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/Readme.java
similarity index 76%
rename from 
quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/package-info.java
rename to 
quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/Readme.java
index 75ce9829..1e9cd889 100644
--- 
a/quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/package-info.java
+++ 
b/quarkus/defaults/src/main/java/org/apache/polaris/service/defaults/Readme.java
@@ -16,9 +16,13 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-
 package org.apache.polaris.service.defaults;
 
-// This file is here as a placeholder to have some java content in
-// the corresponding jar. Relevant data is in the application.properties
-// file at the root of the resources tree.
+/**
+ * This file is here as a placeholder to have some java content in the 
corresponding jar. Relevant
+ * data is in the application.properties file at the root of the resources 
tree.
+ */
+@SuppressWarnings("unused")
+public final class Readme {
+  private Readme() {}
+}
diff --git a/quarkus/server/build.gradle.kts b/quarkus/server/build.gradle.kts
index a082bf34..1fd07546 100644
--- a/quarkus/server/build.gradle.kts
+++ b/quarkus/server/build.gradle.kts
@@ -65,8 +65,6 @@ tasks.named("distZip") { dependsOn("quarkusBuild") }
 
 tasks.named("distTar") { dependsOn("quarkusBuild") }
 
-tasks.withType<Javadoc> { isFailOnError = false }
-
 tasks.register("run") { dependsOn("quarkusRun") }
 
 tasks.named<QuarkusRun>("quarkusRun") {
diff --git 
a/quarkus/server/src/main/java/org/apache/polaris/server/quarkus/package-info.java
 b/quarkus/server/src/main/java/org/apache/polaris/server/quarkus/Readme.java
similarity index 78%
rename from 
quarkus/server/src/main/java/org/apache/polaris/server/quarkus/package-info.java
rename to 
quarkus/server/src/main/java/org/apache/polaris/server/quarkus/Readme.java
index fc12d8ca..5617d530 100644
--- 
a/quarkus/server/src/main/java/org/apache/polaris/server/quarkus/package-info.java
+++ b/quarkus/server/src/main/java/org/apache/polaris/server/quarkus/Readme.java
@@ -16,8 +16,13 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-
 package org.apache.polaris.server.quarkus;
 
-// This file is here as a placeholder to allow quarkusDev and quarkusRun
-// tasks to work, since this module currently has no source files.
+/**
+ * This file is here as a placeholder to allow quarkusDev and quarkusRun tasks 
to work, since this
+ * module currently has no source files.
+ */
+@SuppressWarnings("unused")
+public final class Readme {
+  private Readme() {}
+}

Reply via email to