-
Notifications
You must be signed in to change notification settings - Fork 8
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 scalafix integration #159
Conversation
lazy val scalafixSettings: Seq[Setting[_]] = | ||
Seq( | ||
scalafixDependencies ++= Seq( | ||
"org.http4s" %% "http4s-scalafix-internal" % "0.23.12", | ||
"com.github.liancheng" %% "organize-imports" % "0.6.0" | ||
) | ||
) |
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.
Weird or no?
Actually I think we can upstream many/all of the internal rules to typelevel-scalafix. They seem widely applicable.
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.
"Upstream to Typelevel unless it's HTTP" has been my mantra for many things. Seems applicable here, too.
lazy val scalafixSettings: Seq[Setting[_]] = | ||
Seq( | ||
scalafixDependencies ++= Seq( | ||
"org.http4s" %% "http4s-scalafix-internal" % "0.23.12", | ||
"com.github.liancheng" %% "organize-imports" % "0.6.0" | ||
) | ||
) |
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.
"Upstream to Typelevel unless it's HTTP" has been my mantra for many things. Seems applicable here, too.
Alrighty, let's ship this. |
No description provided.