Brice Jaglin

Results 255 comments of Brice Jaglin

That change _looks_ non-breaking from a Scalafix perspective as ScalaPB classes are not leaking to the public Scalafix API (they are all converted to case classes manually-defined in `scalafix.v1`). When...

You should have been using `scalafixDependencies`, not `libraryDependencies`. In any case, OrganizeImports is now [a built-in rule](https://github.com/scalacenter/scalafix/pull/1480) as of Scalafix 0.11.0 so this should not be required anymore.

A new release will be needed for compatbility with scalafix 0.10.0, see https://github.com/scala/scala-rewrites/pull/153. For the time being, stick to scalafix 0.9.x as mentioned in the README.

This looks like a problem with the sbt-scalafix cache indeed. I haven't been able to reproduce the issue, but if you manage to find a repro, I'll be glad to...

6d8b555147fdab3d001fc24411dabb237bde6512 causes core compilation to succeed, but docs2_13 to fail > ```[error] Caused by: java.lang.ClassNotFoundException: scala.meta.trees.Origin``` That's because for some reason, coursier prefers 4.8.15 over 4.8.15+150-a7c0baa0-SNAPSHOT there: ``` sbt:scalafix> show...

ea2e835ecdb817268bc963a4093d7913daed8498 exposes an incompatibility between mdoc and the latest scalameta SNAPSHOT > ```[error] Caused by: java.lang.ClassNotFoundException: scala.meta.internal.prettyprinters.enquote$``` Following-up in https://github.com/scalameta/mdoc/pull/842

``` [info] warning: patch.md:168:22: local val dialectOnly$macro$4 in value qual$12 is never used [info] Patch.removeImportee(importee"Future").showDiff() [info] ^^^^^^^^ ``` Following-up in https://github.com/scalameta/scalameta/issues/3546

``` [info] error: patch.md:168:1: token not found: [info] Patch.removeImportee(importee"Future").showDiff() [info] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ [info] java.util.NoSuchElementException: token not found: [info] at scalafix.util.TokenList.$anonfun$tok2idx$2(TokenList.scala:27) [info] at scala.collection.immutable.Map$WithDefault.default(Map.scala:181) [info] at scala.collection.MapOps.apply(Map.scala:176) [info] at scala.collection.MapOps.apply$(Map.scala:175) [info] at...

Thanks for reaching out! First of all, it's worth noting that [initial support for RemoveUnused is around the corner, in time for Scala 3.3.0](https://github.com/scalacenter/scalafix/pull/1728), so hopefully this will be a...

Thanks for the report! I am hesitant to add an exception that would effectively change the semantics of `DisableSyntax.noAsInstanceOf`, especially since forcing usage of `asInstanceOf` serves a purpose [per the...