Rename Reflection.{Comment => Documentation}

The name of the concept does not suggest that these are the documentation string.
It also suggests that the comments can be accessed which is not the case.

Author: nicolasstucki

compiler/src/dotty/tools/dotc/quoted/QuoteContextImpl.scala

@@ -2241,11 +2241,11 @@ class QuoteContextImpl private (ctx: Context) extends QuoteContext:
 
         def localContext: Context =
           if self.exists then ctx.withOwner(self) else ctx
-        def comment: Option[Comment] =
+        def documentation: Option[Documentation] =
           import dotc.core.Comments.CommentsContext
           val docCtx = ctx.docCtx.getOrElse {
             throw new RuntimeException(
-              "DocCtx could not be found and comments are unavailable. This is a compiler-internal error."
+              "DocCtx could not be found and documentations are unavailable. This is a compiler-internal error."
             )
           }
           docCtx.docstring(self)
@@ -2528,18 +2528,18 @@ class QuoteContextImpl private (ctx: Context) extends QuoteContext:
     def warning(msg: => String, sourceFile: SourceFile, start: Int, end: Int): Unit =
       dotc.report.error(msg, dotc.util.SourcePosition(sourceFile, dotc.util.Spans.Span(start, end)))
 
-    type Comment = dotc.core.Comments.Comment
+    type Documentation = dotc.core.Comments.Comment
 
-    object Comment extends CommentModule
+    object Documentation extends DocumentationModule
 
-    object CommentMethodsImpl extends CommentMethods:
-      extension (self: Comment):
+    object DocumentationMethodsImpl extends DocumentationMethods:
+      extension (self: Documentation):
         def raw: String = self.raw
         def expanded: Option[String] = self.expanded
         def usecases: List[(String, Option[DefDef])] =
           self.usecases.map { uc => (uc.code, uc.tpdCode) }
       end extension
-    end CommentMethodsImpl
+    end DocumentationMethodsImpl
 
     private def optional[T <: dotc.ast.Trees.Tree[?]](tree: T): Option[tree.type] =
       if tree.isEmpty then None else Some(tree)

library/src/scala/tasty/Reflection.scala

@@ -107,7 +107,7 @@ import scala.tasty.reflect._
  *
  *  +- Position
  *
- *  +- Comment
+ *  +- Documentation
  *
  *  +- Constant
  *
@@ -2575,8 +2575,8 @@ trait Reflection { reflection =>
 
       def localContext: Context
 
-      /** The comment for this symbol, if any */
-      def comment: Option[Comment]
+      /** The documentation for this symbol, if any */
+      def documentation: Option[Documentation]
 
       /** Tree of this definition
         *
@@ -3162,28 +3162,26 @@ trait Reflection { reflection =>
   /** Emits a warning at a specific range of a file */
   def warning(msg: => String, source: SourceFile, start: Int, end: Int): Unit
 
-  //////////////
-  // COMMENTS //
-  //////////////
-
-  // TODO: misnomer. Rename to `Documentation`
+  ///////////////////
+  // DOCUMENTATION //
+  ///////////////////
 
   /** Attachment representing the documentation of a definition */
-  type Comment <: AnyRef
+  type Documentation <: AnyRef
 
-  val Comment: CommentModule
+  val Documentation: DocumentationModule
 
-  trait CommentModule { this: Comment.type => }
+  trait DocumentationModule { this: Documentation.type => }
 
-  given CommentMethods as CommentMethods = CommentMethodsImpl
-  protected val CommentMethodsImpl: CommentMethods
+  given DocumentationMethods as DocumentationMethods = DocumentationMethodsImpl
+  protected val DocumentationMethodsImpl: DocumentationMethods
 
-  trait CommentMethods {
-    extension (self: Comment):
-      /** Raw comment string */
+  trait DocumentationMethods {
+    extension (self: Documentation):
+      /** Raw documentation string */
       def raw: String
 
-      /** Expanded comment string, if any */
+      /** Expanded documentation string, if any */
       def expanded: Option[String]
 
       /** List of usecases and their corresponding trees, if any */