scalafix has to use -Ywarn-

Hi <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url="

Hi, that suggestion does not work for me. I still get this error: <div class="snip

`sbt-tpolecat` is conflict with `scalafix` about sbt-tpolecat HOT 3 CLOSED

typelevel commented on August 14, 2024

`sbt-tpolecat` is conflict with `scalafix`

from sbt-tpolecat.

Comments (3)

DavidGregory084 commented on August 14, 2024

Hi @UkonnRa, a lot of users use sbt-tpolecat because it ensures that code with warnings can't be committed. Could you potentially use something like the following to resolve this conflict?

scalacOptions.in(Compile, scalafix) ~= { options: Seq[String] =>
  options.filterNot(Set("-Werror"))
}

scalacOptions.in(Test, scalafix) ~= { options: Seq[String] =>
  options.filterNot(Set("-Werror"))
}

from sbt-tpolecat.

UkonnRa commented on August 14, 2024

Cool! THX a lot!

from sbt-tpolecat.

mrt181 commented on August 14, 2024

Hi, that suggestion does not work for me. I still get this error:

sbt> scalafix RemoveUnused
[error] import java.util.UUID                             
[error]                  ^                                
[error] No warnings can be incurred under -Werror.        
[error] two errors found                                  
[error] (Compile / compileIncremental) Compilation failed

I have to set -Ywarn-unused:imports explicitly otherwise I get an error that at least one -Ywarn-unused setting must be set

lazy val settings = 
  Seq(
    scalacOptions += "-Ymacro-annotations",
    scalacOptions += "-Ywarn-unused:imports",
    scalacOptions.in(Compile, scalafix) ~= { options: Seq[String] =>
      options.filterNot(Set("-Werror"))
    },
    scalacOptions.in(Test, scalafix) ~= { options: Seq[String] =>
      options.filterNot(Set("-Werror"))
    }
  )