From 5df1611ee3b59f6c20cf9eb7eeec60bc9d9e8cdb Mon Sep 17 00:00:00 2001 From: Olger Warnier Date: Tue, 12 May 2020 22:13:24 +0200 Subject: [PATCH 1/2] Fixed TableMigration Reversible class to have an explicit toString in order to be sure that description comparisons in the flyway extension work as expected. --- src/main/scala/slick/migration/api/TableMigration.scala | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/main/scala/slick/migration/api/TableMigration.scala b/src/main/scala/slick/migration/api/TableMigration.scala index 8121d3dc..2d798daf 100644 --- a/src/main/scala/slick/migration/api/TableMigration.scala +++ b/src/main/scala/slick/migration/api/TableMigration.scala @@ -84,6 +84,9 @@ object TableMigration { } } } + override def toString: String = { + "Reversible " + this.underlying.toString + } } implicit def toReversible[T <: JdbcProfile#Table[_]]: ToReversible[TableMigration[T, Action.Reversible]] = From 1954bd7d051911c98cdbc29fe774431f87fc64ac Mon Sep 17 00:00:00 2001 From: Olger Warnier Date: Tue, 19 May 2020 14:45:55 +0200 Subject: [PATCH 2/2] Change based on review comments --- src/main/scala/slick/migration/api/TableMigration.scala | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/main/scala/slick/migration/api/TableMigration.scala b/src/main/scala/slick/migration/api/TableMigration.scala index 2d798daf..de1a1d47 100644 --- a/src/main/scala/slick/migration/api/TableMigration.scala +++ b/src/main/scala/slick/migration/api/TableMigration.scala @@ -84,9 +84,7 @@ object TableMigration { } } } - override def toString: String = { - "Reversible " + this.underlying.toString - } + override def toString = underlying.toString } implicit def toReversible[T <: JdbcProfile#Table[_]]: ToReversible[TableMigration[T, Action.Reversible]] =