-
Notifications
You must be signed in to change notification settings - Fork 594
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
Akka HTTP BOM #3665
Akka HTTP BOM #3665
Conversation
Test PASSed. |
build.sbt
Outdated
@@ -46,6 +46,25 @@ inThisBuild(Def.settings( | |||
scalafixScalaBinaryVersion := scalaBinaryVersion.value, | |||
)) | |||
|
|||
// When this is updated the set of modules in ActorSystem.allModules should also be updated |
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.
// When this is updated the set of modules in ActorSystem.allModules should also be updated |
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.
I accepted your suggestion and then realized the comment is required (but was wrong). It should read In Http.allModules
.
Test PASSed. |
Co-authored-by: Arnout Engelen <github@bzzt.net>
Test PASSed. |
Test PASSed. |
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.
LGTM (I can't approve because I'm the original author of the PR)
We need to settle the discussion in akka/akka#29839 (comment) first. |
Test PASSed. |
project/plugins.sbt
Outdated
@@ -17,6 +17,7 @@ addSbtPlugin("net.virtual-void" % "sbt-dependency-graph" % "0.10.0-RC1") // for | |||
addSbtPlugin("io.spray" % "sbt-boilerplate" % "0.6.1") | |||
addSbtPlugin("org.foundweekends" % "sbt-bintray" % "0.6.1") | |||
addSbtPlugin("com.lightbend.sbt" % "sbt-publish-rsync" % "0.2") | |||
addSbtPlugin("com.lightbend.sbt" % "sbt-bill-of-materials" % "1.0.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.
🎉
Test PASSed. |
@@ -1,11 +1,11 @@ | |||
import akka._ | |||
import akka.ValidatePullRequest._ | |||
import AkkaDependency._ | |||
import Dependencies.{ h2specName, h2specExe } | |||
import Dependencies.{h2specExe, h2specName} |
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.
surprised this didn't trigger scalariform - I guess it doesn't check the build.sbt?
@@ -23,7 +23,7 @@ inThisBuild(Def.settings( | |||
scmInfo := Some( | |||
ScmInfo(url("https://github.com/akka/akka-http"), "git@github.com:akka/akka-http.git")), | |||
developers := List( | |||
Developer("contributors", "Contributors", "akka-user@googlegroups.com", | |||
Developer("contributors", "Contributors", "info@lightbend.com", |
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.
😆 I'm not sure that would reach us - but akka-user is obviously wrong as well.
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.
akka.official@gmail.com would reach us though
Refs playframework/playframework#10549 and akka/akka#29839