Skip to content

Commit

Permalink
Merge pull request playframework#581 from scala-steward/update/scalaf…
Browse files Browse the repository at this point in the history
…mt-core-2.7.5

Update scalafmt-core to 2.7.5
  • Loading branch information
mergify[bot] committed May 14, 2021
2 parents f60c5c3 + eb372e5 commit d4ecda8
Show file tree
Hide file tree
Showing 9 changed files with 19 additions and 19 deletions.
4 changes: 1 addition & 3 deletions .scalafmt.conf
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,4 @@ rewrite.rules = [ AvoidInfix, ExpandImportSelectors, RedundantParens, SortModifi
rewrite.sortModifiers.order = [ "private", "protected", "final", "sealed", "abstract", "implicit", "override", "lazy" ]
spaces.inImportCurlyBraces = true # more idiomatic to include whitepsace in import x.{ yyy }
trailingCommas = preserve
# `edition` introduced in https://github.com/scalameta/scalafmt/pull/1431
edition = 2019-10
version = 2.5.3
version = 2.7.5
4 changes: 2 additions & 2 deletions docs/manual/working/scalaGuide/main/sql/slick/code/DI.scala
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ import slick.jdbc.JdbcProfile
import UsersSchema._

//#di-database-config
class Application @Inject() (protected val dbConfigProvider: DatabaseConfigProvider, cc: ControllerComponents)(
implicit ec: ExecutionContext
class Application @Inject() (protected val dbConfigProvider: DatabaseConfigProvider, cc: ControllerComponents)(implicit
ec: ExecutionContext
) extends AbstractController(cc)
with HasDatabaseConfigProvider[JdbcProfile] {
//#di-database-config
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,8 @@ import scala.concurrent.ExecutionContext
import scala.concurrent.Future
import scalaguide.slick.UsersSchema._

class Application1 @Inject() (protected val dbConfigProvider: DatabaseConfigProvider, cc: ControllerComponents)(
implicit ec: ExecutionContext
class Application1 @Inject() (protected val dbConfigProvider: DatabaseConfigProvider, cc: ControllerComponents)(implicit
ec: ExecutionContext
) extends AbstractController(cc)
with HasDatabaseConfigProvider[JdbcProfile] {

Expand Down
2 changes: 1 addition & 1 deletion project/Dependencies.scala
Original file line number Diff line number Diff line change
Expand Up @@ -31,5 +31,5 @@ object Library {
val playSpecs2 = "com.typesafe.play" %% "play-specs2" % Version.play
val slick = "com.typesafe.slick" %% "slick" % Version.slick
val slickHikariCP = "com.typesafe.slick" %% "slick-hikaricp" % Version.slick
val h2 = "com.h2database" % "h2" % Version.h2
val h2 = "com.h2database" % "h2" % Version.h2
}
Original file line number Diff line number Diff line change
Expand Up @@ -193,8 +193,8 @@ trait HasDatabaseConfig[P <: BasicProfile] {
*/
trait HasDatabaseConfigProvider[P <: BasicProfile] extends HasDatabaseConfig[P] {

/** The provider of a Slick `DatabaseConfig` instance.*/
/** The provider of a Slick `DatabaseConfig` instance. */
protected val dbConfigProvider: DatabaseConfigProvider
protected final override lazy val dbConfig
: DatabaseConfig[P] = dbConfigProvider.get[P] // field is lazy to avoid early initializer problems.
protected final override lazy val dbConfig: DatabaseConfig[P] =
dbConfigProvider.get[P] // field is lazy to avoid early initializer problems.
}
4 changes: 2 additions & 2 deletions src/core/src/main/scala/play/api/db/slick/SlickApi.scala
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,8 @@ object DefaultSlickApi {
private val logger = Logger(classOf[DefaultSlickApi])
}
// This class is useful for delaying the creation of `DatabaseConfig` instances.
private class DatabaseConfigFactory(name: String, config: Config, lifecycle: ApplicationLifecycle)(
implicit executionContext: ExecutionContext
private class DatabaseConfigFactory(name: String, config: Config, lifecycle: ApplicationLifecycle)(implicit
executionContext: ExecutionContext
) {
import DatabaseConfigFactory.logger

Expand Down
8 changes: 5 additions & 3 deletions src/core/src/main/scala/play/api/db/slick/SlickModule.scala
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,10 @@ import scala.concurrent.ExecutionContext

object SlickModule {

/** path in the **reference.conf** to obtain the path under which databases are configured.*/
/** path in the **reference.conf** to obtain the path under which databases are configured. */
final val DbKeyConfig = "play.slick.db.config"

/** path in the **reference.conf** to obtain the name of the default database.*/
/** path in the **reference.conf** to obtain the name of the default database. */
final val DefaultDbName = "play.slick.db.default"
}

Expand All @@ -36,7 +36,9 @@ final class SlickModule extends Module {
val dbKey = config.getString(SlickModule.DbKeyConfig)
val default = config.getString(SlickModule.DefaultDbName)
val dbs = configuration.getOptional[Configuration](dbKey).getOrElse(Configuration.empty).subKeys
Seq(bind[SlickApi].to[DefaultSlickApi].in[Singleton]) ++ namedDatabaseConfigBindings(dbs) ++ defaultDatabaseConfigBinding(
Seq(bind[SlickApi].to[DefaultSlickApi].in[Singleton]) ++ namedDatabaseConfigBindings(
dbs
) ++ defaultDatabaseConfigBinding(
default,
dbs
)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class SlickApiSpec extends Specification {
"SlickApi.dbConfigs" should {
"return all DatabaseConfig instances for a valid configuration" in {
import SUTWithGoodConfig._
api.dbConfigs[BasicProfile] must have size (4)
api.dbConfigs[BasicProfile] must have size 4
}
"throw if a database name doesn't exist in the config" in {
import SUTWithBadConfig._
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,8 +21,8 @@ import scala.util.control.ControlThrowable
private[evolutions] class DBApiAdapter @Inject() (slickApi: SlickApi) extends DBApi {
private lazy val databasesByName: Map[DbName, PlayDatabase] = slickApi
.dbConfigs[JdbcProfile]()
.map {
case (name, dbConfig) => (name, new DBApiAdapter.DatabaseAdapter(name, dbConfig))
.map { case (name, dbConfig) =>
(name, new DBApiAdapter.DatabaseAdapter(name, dbConfig))
}
.toMap

Expand Down

0 comments on commit d4ecda8

Please sign in to comment.