Scala.js options overwritten on Scala 3 about sbt-tpolecat HOT 8 CLOSED

Sorry for the silence on this one folks - I'm going mad trying to find a way to solve this without breaking any of the previous behaviours of the library and while still accomodating the ways that folks set the various keys of the library. The combination of +=, ++= and sbt scope delegation is very hard to navigate.

from sbt-tpolecat.

Sorry for adding more salt to the wound, but I just found that sbt-tpolecat also causes problems with metals / semanticdb; where this plugin ends up removing the required scalac flags for it to work properly.

Apparently, the issue is related to the exclusions rule.
I didn't test with main sources, but if I do this:

Test / tpolecatExcludeOptions ++= ScalacOptions.defaultConsoleExclude

Then metals is broken on the test files.

from sbt-tpolecat.

PR available: #126

from sbt-tpolecat.

I just stumbled across this issue via this project ( github.com/objektwerks/scalajs/blob/master/build.sbt ). To avoid getting this error during sbt clean test ( java.lang.AssertionError: assertion failed: asTerm called on not-a-Term val ), I had to add scalacOptions ++= Seq("-scalajs") to the build.sbt. Apparently sbt-tpolecat was removing the -scalajs option added by the Scalajs plugin ( addSbtPlugin("org.scala-js" % "sbt-scalajs" % "1.10.1") ).

from sbt-tpolecat.

Unfortunately the obvious fix to this - appending the existing scalacOptions.value to the resulting scalacOptions after this plugin is applied - doesn't resolve the issue, or at least it causes more issues.
I've started a discussion in sbt (sbt/sbt#6986) about how to resolve this so hopefully we will get some good advice there.

from sbt-tpolecat.

Beyond simple tasks, Sbt is like an extreme sport. So be careful. :)

from sbt-tpolecat.

Just bumping this issue, it only occurred for me when I tried to go from sbt-tpolecat version 0.3.1 to 0.4.1. Same output as originally shown:

[info] Running scalafix on 1 Scala sources (incremental)
[warn] The `-scalajs` flag was missing from `compile / scalacOptions`, but it is required to produce Scala.js IR.
[warn] Linking, running and/or testing will probably go wrong.
[warn] The most likely cause is that you used `scalacOptions := ...` instead of using `++=`.

from sbt-tpolecat.

I can confirm that reproducer no longer works with a locally published version of the plugin 🎉

from sbt-tpolecat.