olabusayoT commented on code in PR #1479:
URL: https://github.com/apache/daffodil/pull/1479#discussion_r2190845310


##########
build.sbt:
##########
@@ -456,59 +409,39 @@ lazy val ratSettings = Seq(
 def apiDocSourceFilter(sources: Seq[File]): Seq[File] = sources.filter { 
source =>
   val str = source.toString
   val oad = "/org/apache/daffodil"
-  lazy val excludedForJAPI =
-    str.contains(oad + "/japi/") && {
-      // Some things are excluded from the JAPI javadoc because they are 
internal, non-API
-      // These cannot be excluded from SAPI because symbols-not-found issues 
with scaladoc.
-      str.contains("$") || str.contains("packageprivate")
-    }
   lazy val included = {
     str.contains(oad + "/udf/") ||
-    str.contains(oad + "/sapi/") ||
-    str.contains(oad + "/japi/") ||
+    str.contains(oad + "/api/") ||
     str.contains(oad + "/runtime1/layers/api/")
     //
     // There are files in runtime1/api that are NOT part of the public, 
supported API.
     // I tried to include all of runtime1/api, and exclude those files, but 
could not
     // get that to work, so now we include individually each file that is part 
of the
     // published runtime1 API
-    //
-    // NOTE: Commented out for now. genjavadoc doesn't handle the traits in
-    // these files, so for now these are undocumented.
-    //
-    // FIXME: DAFFODIL-2902
-    //    str.contains(oad + "/runtime1/api/DFDLPrimType") ||
-    //    str.contains(oad + "/runtime1/api/Infoset") ||
-    //    str.contains(oad + "/runtime1/api/Metadata")
   }
-  val res = included && !excludedForJAPI
+  val res = included
   res
 }
 
 lazy val unidocSettings =

Review Comment:
   see https://github.com/apache/daffodil-site/pull/183



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to