bazelbuild / rules_scala

Scala rules for Bazel
Apache License 2.0
363 stars 278 forks source link

Compiler diagnostics for scala 3 are missing. #1602

Closed ValdemarGr closed 1 month ago

ValdemarGr commented 3 months ago

The current Scala 3 diagnostics reporter looks like it is a noop.

It looks like the scala 3 compiler has a ConsoleReporter here. Are there any obstacles for implementing it like the scala 2 variant?

WojciechMazur commented 2 months ago

I've started to work on that, but I cannot assign myself to the issue. I'm getting strange cyclic inheritance errors when extending 'dotc.reporting.ConsoleReporter'. Might need to check if Scala compiler does not produce something strange in the byte code

susliko commented 2 months ago

@WojciechMazur I also was working on this today, haven't noticed your comment 😓

Looks like I've got it working, but there are no tests yet.

It would be great to join efforts on this.

strange cyclic inheritance errors when extending 'dotc.reporting.ConsoleReporter'.

I chose composition over inheritance 🙂