Hello, I have a pull request to upgrade scalafix-maven-plugin version [1]. I hope it will be approvable by the maintainer. I also have another request to James [2], that implement scalafix tool.
[1] https://github.com/evis/scalafix-maven-plugin/pull/15 [2] https://github.com/apache/james-project/pull/538 Cheers, Tung On Tue, Jul 13, 2021 at 6:59 PM Jean Helou <jean.he...@gmail.com> wrote: > > Yes, I did. I forked this plugin and manual change scala version on > local. > > As I wrote > > > > sorry I didn't catch that in your original email > > > "I got another issue: java.lang.NoSuchMethodError: > > scala.reflect.internal.util.ReusableInstance, > > scala/collection/IterableOnce" > > Maybe need more time to check. > > > > just a thought but the maven plugin hasn't been updated in a while and > pulls scalafix 0.9.23 > https://github.com/scalacenter/scalafix/releases/tag/v0.9.26 mentions > scala > 2.13.5 with link to a PR[1] which explicitely mentions the rule you are > trying to use :) > > can you try upgrading scalafix version from 9.23 at least to 9.26 (even > better to 9.29 in your local build ? > > [1] https://github.com/scalacenter/scalafix/pull/1334/files > > cheers, > jean > > Thank you, > > > > On Tue, Jul 13, 2021 at 6:34 PM Jean Helou <jean.he...@gmail.com> wrote: > > > > > This is my configuration > > > > > > > > > > looks good to me, so it looks like the plugin is indeed pinned on scala > > > 2.12, you could clone the repo, change the plugin version to a custom > > > version, change references to scala 2.12 to scala 2.13 and run an > install > > > locally to check that having the plugin itself compiled against 2.13 > > works. > > > if it does you can chime in on the PR to try and get it merged in an > > > official manner. > > > What I usually do in such cases is publish an unofficial version of the > > > tool on an unofficial repo and use that until the PR is merged. not > sure > > > how that flies for apache projects though especially since the plugin > > uses > > > a BSD3 license. > > > Since it's not a published artifact of the project maybe we are allowed > > to > > > do that as long as we publish the custom build to a public repo ? I > used > > to > > > publish on bintray not sure where to go now .. > > > > > > Regards, > > > Jean > > > > > > > > > > On Tue, Jul 13, 2021 at 5:21 PM Jean Helou <jean.he...@gmail.com> > > wrote: > > > > > > > > > Hi Tung, > > > > > > > > > > > > > > > > Very interesting with the rewrite feature of the scalafix tool. > > > > > > I tried it for the `jmap-rfc-8621` module. I got an incompatible > > > issue > > > > > with > > > > > > the scala version when trying Semantic Rules. > > > > > > > > > > > > > > > > According to the issue you link to, the plugin only works with > scala > > > 2.12 > > > > > and according to the error message you report, the james code base > > uses > > > > > scala 2.13. > > > > > In this case it won't work unless the plugin is cross compiled for > > > 2.13. > > > > > What's intriguing is that the tips and tricks section [1] has a > > sample > > > > > configuration for the semantic db compiler which seems ot hint > that > > > the > > > > > scala.version can be configured. but i actually doubt it would work > > > > > > > > > > can you share the configuration you used ? > > > > > [1] https://github.com/evis/scalafix-maven-plugin#tips-and-tricks > > > > > > > > > > I don't use the maven plugin itself, all my scala projects use > either > > > SBT > > > > > or mill so I don't have this layer of issues. > > > > > > > > > > Regards, > > > > > Jean > > > > > > > > > > [1] https://github.com/evis/scalafix-maven-plugin/pull/12 > > > > > > > > > > > > Regards, > > > > > > Tung > > > > > > > > > > > > On Fri, Jun 25, 2021 at 2:44 PM Jean Helou <jean.he...@gmail.com > > > > > > wrote: > > > > > > > > > > > > > Hi tung, > > > > > > > > > > > > > > Have you looked at scalafix[1]? it seems to be much more > actively > > > > > > > maintained than scalastyle and I feel that it's scope overlaps > > > > > > scalatyle's > > > > > > > quite a bit... > > > > > > > > > > > > > > [1]https://github.com/scalacenter/scalafix > > > > > > > > > > > > > > Regards, > > > > > > > jean > > > > > > > > > > > > > > > > > > > > > On Fri, Jun 25, 2021 at 8:14 AM tungtv...@gmail.com < > > > > > tungtv...@gmail.com > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > Hello guys, > > > > > > > > > > > > > > > > I realize the current checkstyle.xml file doesn't support the > > > Scala > > > > > > > > convention. > > > > > > > > Should we have a checkstyle for that? > > > > > > > > > > > > > > > > My suggestion: > > > > > > > > - https://github.com/scalastyle/scalastyle > > > > > > > > > > > > > > > > - https://github.com/scalastyle/scalastyle-maven-plugin > > > > > > > > > > > > > > > > > > > > > > > > Best Regards, > > > > > > > > > > > > > > > > Tung, Van TRAN > > > > > > > > > > > > > > > > > > > > > > > > > > > > --------------------------------------------------------------------- > > > > > > > > To unsubscribe, e-mail: > > server-dev-unsubscr...@james.apache.org > > > > > > > > For additional commands, e-mail: > > > server-dev-h...@james.apache.org > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Tung, Tran Van > > > > > > *Phone:* (+84) 35 757 6258 > > > > > > *Skype:* tung.tv202 > > > > > > > > > > > > > > > > > > > > > > > -- > > > > Tung, Tran Van > > > > *Phone:* (+84) 35 757 6258 > > > > *Skype:* tung.tv202 > > > > > > > > > > > > > -- > > Tung, Tran Van > > *Phone:* (+84) 35 757 6258 > > *Skype:* tung.tv202 > > > -- Tung, Tran Van *Phone:* (+84) 35 757 6258 *Skype:* tung.tv202
