-
Notifications
You must be signed in to change notification settings - Fork 185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Mega-Linter in list of integrating tools #1312
Conversation
Thanks for the integration @nvuillam. One gotcha is that - as far as I can see - you are running |
Thanks for your reply :) Is there a way to generically perform scalax compilation and semanticdb generation via cli ?
|
|
Updated ! Is is ok like that ? https://nvuillam.github.io/mega-linter/descriptors/scala_scalafix/ |
I also updated my PR (online in github editor, so you may use Squash & merge option to avoid useless commits in master ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A suggestion for the intro of https://nvuillam.github.io/mega-linter/descriptors/scala_scalafix/ focusing on the user-facing limitations
Scalafix lints and fixes scala files
- Semantic rules are not supported
- External rules are not supported
Co-authored-by: Brice Jaglin <bjaglin@gmail.com>
as you wish :) |
Hi, thanks for your linter :)
I integrated it in Mega-Linter, can I put a link here ? :)
Internal documentation is https://nvuillam.github.io/mega-linter/descriptors/scala_scalafix/, do you have remarks ?
Best regards :)