Comments.

src/main/kotlin/com/beust/kobalt/api/BasePlugin.kt

@@ -12,6 +12,9 @@ abstract public class BasePlugin : IPlugin {
         this.context = context
     }
 
+    /**
+     * The list of projects depended upon (e.g. val p = javaProject(dependentProject)).
+     */
     protected val projects = arrayListOf<ProjectDescription>()
 
     fun addProject(project: Project, dependsOn: Array<out Project>) =

src/main/kotlin/com/beust/kobalt/maven/DependencyManager.kt

@@ -39,6 +39,10 @@ public class DependencyManager @Inject constructor(val executors: KobaltExecutor
         return result
     }
 
+    /**
+     * Return the transitive closure of the dependencies *without* running the classpath contributors.
+     * TODO: This should be private, everyone should be calling calculateDependencies().
+     */
     fun transitiveClosure(dependencies : List<IClasspathDependency>): List<IClasspathDependency> {
         var executor = executors.newExecutor("JvmCompiler}", 10)
 

src/main/kotlin/com/beust/kobalt/plugin/kotlin/KotlinCompiler.kt

@@ -107,6 +107,8 @@ class KotlinCompiler @Inject constructor(val localRepo : LocalRepo,
 
     /**
      * Create an ICompilerAction based on the parameters and send it to JvmCompiler.doCompile().
+     * TODO: This needs to be removed because it doesn't use contributors. Call
+     * JvmCompilerPlugin#createCompilerActionInfo instead
      */
     fun compile(project: Project?, context: KobaltContext?, compileDependencies: List<IClasspathDependency>,
             otherClasspath: List<String>, sourceFiles: List<String>, outputDir: File, args: List<String>) : TaskResult {