Address review comments

Author: Duhemm

compiler/src/dotty/tools/dotc/sbt/ExtractAPI.scala

@@ -254,24 +254,25 @@ private class ExtractAPICollector(implicit val ctx: Context) extends ThunkHolder
   def apiClassStructure(csym: ClassSymbol): api.Structure = {
     val cinfo = csym.classInfo
 
-    val bases =
-      try {
-        val ancestorTypes0 = linearizedAncestorTypes(cinfo)
-        if (ValueClasses.isDerivedValueClass(csym)) {
-          val underlying = ValueClasses.valueClassUnbox(csym).info.finalResultType
-          // The underlying type of a value class should be part of the name hash
-          // of the value class (see the test `value-class-underlying`), this is accomplished
-          // by adding the underlying type to the list of parent types.
-          underlying :: ancestorTypes0
-        } else
-          ancestorTypes0
-      } catch {
-        case ex: CyclicReference =>
-          // See neg/i1750a for an example where a cyclic error can arise.
-          // The root cause in this example is an illegal "override" of an inner trait
-          ctx.error(cyclicErrorMsg(ex), csym.pos)
-          defn.ObjectType :: Nil
-       }
+    val bases = {
+      val ancestorTypes0 =
+        try linearizedAncestorTypes(cinfo)
+        catch {
+          case ex: CyclicReference =>
+            // See neg/i1750a for an example where a cyclic error can arise.
+            // The root cause in this example is an illegal "override" of an inner trait
+            ctx.error(cyclicErrorMsg(ex), csym.pos)
+            defn.ObjectType :: Nil
+        }
+      if (ValueClasses.isDerivedValueClass(csym)) {
+        val underlying = ValueClasses.valueClassUnbox(csym).info.finalResultType
+        // The underlying type of a value class should be part of the name hash
+        // of the value class (see the test `value-class-underlying`), this is accomplished
+        // by adding the underlying type to the list of parent types.
+        underlying :: ancestorTypes0
+      } else
+        ancestorTypes0
+    }
 
     val apiBases = bases.map(apiType)
 

project/Build.scala

@@ -197,8 +197,8 @@ object Build {
     libraryDependencies ++= {
       if (bootstrapFromPublishedJars.value)
         Seq(
-          dottyOrganization % "dotty-library_2.11" % dottyNonBootstrappedVersion % Configurations.ScalaTool.name,
-          dottyOrganization % "dotty-compiler_2.11" % dottyNonBootstrappedVersion % Configurations.ScalaTool.name
+          dottyOrganization %% "dotty-library" % dottyNonBootstrappedVersion % Configurations.ScalaTool.name,
+          dottyOrganization %% "dotty-compiler" % dottyNonBootstrappedVersion % Configurations.ScalaTool.name
         ).map(_.withDottyCompat(scalaVersion.value))
       else
         Seq()
@@ -862,7 +862,7 @@ object Build {
       unmanagedSourceDirectories in Compile +=
         baseDirectory.value / "../language-server/src/dotty/tools/languageserver/config",
       sbtPlugin := true,
-      version := "0.1.6-SNAPSHOT",
+      version := "0.1.8",
       sbtTestDirectory := baseDirectory.value / "sbt-test",
       scriptedLaunchOpts += "-Dplugin.version=" + version.value,
       scriptedLaunchOpts += "-Dplugin.scalaVersion=" + dottyVersion,