Quote version string in replaceAllIn

... so that version ranges are matched literally.

closes #26

Author: fthomas

modules/core/src/main/scala/eu/timepit/scalasteward/github.scala

@@ -44,7 +44,7 @@ object github {
           "--data",
           s"""{
              |  "title": "${localUpdate.commitMsg}",
-             |  "body": "Update ${localUpdate.update.groupId}:${localUpdate.update.artifactId} from ${localUpdate.update.currentVersion} to ${localUpdate.update.nextVersion}.",
+             |  "body": "Updates ${localUpdate.update.groupId}:${localUpdate.update.artifactId} from ${localUpdate.update.currentVersion} to ${localUpdate.update.nextVersion}.",
              |  "head": "$myLogin:${localUpdate.updateBranch.name}",
              |  "base": "${localUpdate.localRepo.base.name}"
              |}

modules/core/src/main/scala/eu/timepit/scalasteward/model/Update.scala

@@ -19,6 +19,8 @@ package eu.timepit.scalasteward.model
 import cats.data.NonEmptyList
 import cats.implicits._
 
+import scala.util.matching.Regex
+
 final case class Update(
     groupId: String,
     artifactId: String,
@@ -47,11 +49,12 @@ final case class Update(
 
   def replaceAllIn(str: String): Option[String] = {
     def normalize(searchTerm: String): String =
-      Update
-        .removeIgnorableSuffix(searchTerm)
+      Regex
+        .quoteReplacement(Update.removeIgnorableSuffix(searchTerm))
         .replace("-", ".?")
 
-    val regex = s"(?i)(${normalize(name)}.*?)$currentVersion".r
+    val regex =
+      s"(?i)(${normalize(name)}.*?)${Regex.quote(currentVersion)}".r
     var updated = false
     val result = regex.replaceAllIn(str, m => {
       updated = true

modules/core/src/main/scala/eu/timepit/scalasteward/steward.scala

@@ -161,6 +161,7 @@ object steward extends IOApp {
           (true, s"$pr is behind ${baseBranch.name}")
         else
           (false, s"$pr is up-to-date with ${baseBranch.name}")
+        // TODO: Only reset PRs that are still open.
       }
       _ <- log.printInfo(msg)
     } yield result

modules/core/src/test/scala/eu/timepit/scalasteward/model/UpdateTest.scala

@@ -99,6 +99,13 @@ class UpdateTest extends FunSuite with Matchers {
       .replaceAllIn(original) shouldBe Some(expected)
   }
 
+  test("replaceAllIn: version range") {
+    val original = """Seq("org.specs2" %% "specs2-core" % "3.+" % "test")"""
+    val expected = """Seq("org.specs2" %% "specs2-core" % "4.3.4" % "test")"""
+    Update("org.specs2", "specs2-core", "3.+", Nel.of("4.3.4"))
+      .replaceAllIn(original) shouldBe Some(expected)
+  }
+
   test("isImpliedBy") {
     val update0 = Update("org.specs2", "specs2-core", "3.9.4", Nel.of("3.9.5"))
     val update1 = update0.copy(artifactId = "specs2-scalacheck")