Merge pull request #24 from oyvindberg/warnings-instead-of-errors

Add support for configuring the plugin to only emit warnings

acyclic/src/acyclic/plugin/Plugin.scala

@@ -11,16 +11,21 @@ class TestPlugin(val global: Global,
   val name = "acyclic"
 
   var force = false
+  var fatal = true
+
   // Yeah processOptions is deprecated but keep using it anyway for 2.10.x compatibility
   override def processOptions(options: List[String], error: String => Unit): Unit = {
     if (options.contains("force")) {
       force = true
     }
+    if (options.contains("warn")) {
+      fatal = false
+    }
   }
   val description = "Allows the developer to prohibit inter-file dependencies"
 
 
   val components = List[tools.nsc.plugins.PluginComponent](
-    new PluginPhase(this.global, cycleReporter, force)
+    new PluginPhase(this.global, cycleReporter, force, fatal)
   )
 }

acyclic/src/acyclic/plugin/PluginPhase.scala

@@ -17,7 +17,8 @@ import tools.nsc.plugins.PluginComponent
  */
 class PluginPhase(val global: Global,
                   cycleReporter: Seq[(Value, SortedSet[Int])] => Unit,
-                  force: => Boolean)
+                  force: => Boolean,
+                  fatal: => Boolean)
                   extends PluginComponent
                   with GraphAnalysis { t =>
 
@@ -148,7 +149,13 @@ class PluginPhase(val global: Global,
           cycleInfo.map{ case (a, b) => a -> b.map(_.pos.line).to(SortedSet)}
         )
 
-        global.error("Unwanted cyclic dependency")
+        val msg = "Unwanted cyclic dependency"
+        if (fatal) {
+          global.error(msg)
+        } else {
+          global.warning(msg)
+        }
+
         for (Seq((value, locs), (nextValue, _)) <- (cycleInfo :+ cycleInfo.head).sliding(2)){
           global.inform("")
           value match{

readme.md

@@ -220,6 +220,15 @@ import acyclic.skipped
 
 to tell the acyclic plugin to ignore them.
 
+Warnings instead of errors
+==========================
+
+If you want the plugin to only emit warnings instead of errors, add `warn` to the plugin's flags.
+
+```scala
+scalacOptions += "-P:acyclic:warn"
+```
+
 Limitations
 ===========