Skip to content

Commit 28b9f68

Browse files
committed
Rework log messages
1 parent 3808560 commit 28b9f68

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

changelog/ChangelogChecker.scala

+7-9
Original file line numberDiff line numberDiff line change
@@ -25,20 +25,19 @@ class ChangelogChecker(config: Config, versionExceptions: Seq[String], overwrite
2525
).call(stderr = os.Pipe) // mute warnings
2626

2727
private def checkTree(moduleName: String)(binaryVersion: String, previousSnapshot: Dep, currentSnapshot: Dep): Unit =
28-
println(s"\nChecking tree for $binaryVersion...")
28+
println(s"\nChecking tree for binary version $binaryVersion...")
2929

3030
val summary = SnapshotDiffValidator.summarize(previousSnapshot, currentSnapshot)
3131
if summary.illegalDiffs.nonEmpty then
32-
println(s"For $binaryVersion found diffs illegal to introduce on this update type: ${summary.updateType}")
33-
println(s"Illegal diffs: \n${summary.illegalDiffs.mkString(" - ", "\n - ", "\n")}")
32+
println(s"Illegal diffs for ${summary.updateType}: \n${summary.illegalDiffs.mkString(" - ", "\n - ", "\n")}")
3433
if versionExceptions.contains(currentSnapshot.version.toString) then
35-
println(currentSnapshot.version.toString + " is explicit exception, ignoring illegal diffs")
34+
println("Ignoring illegal diffs because " + currentSnapshot.version.toString + " is found in exceptions.txt")
3635
else throw IllegalDiffs()
3736

3837
val diffsFound = summary.diffs.nonEmpty
3938

4039
if diffsFound then
41-
println("Found diffs in development version: \n" + summary.diffs.mkString(" - ", "\n - ", ""))
40+
println("All diffs are: \n" + summary.diffs.mkString(" - ", "\n - ", ""))
4241
else
4342
println("No diffs found")
4443

@@ -55,9 +54,8 @@ class ChangelogChecker(config: Config, versionExceptions: Seq[String], overwrite
5554
def changelogsDiff = generatedChangelog.diff(parsedChangelog)
5655

5756
if changelogsDiff.nonEmpty then
58-
println("Found diffs in changelog: \n" + changelogsDiff.get)
57+
println("Changelog file is outdated: \n" + changelogsDiff.get)
5958
if !overwrite then throw AlreadyExists()
60-
6159
else
6260
val input = if yes then "y" else StdIn.readLine("Overwrite symbols? (y/N)")
6361
if input == "y" || input == "Y" then
@@ -70,10 +68,10 @@ class ChangelogChecker(config: Config, versionExceptions: Seq[String], overwrite
7068
println("Overwritten")
7169
else throw UserRejection()
7270
else
73-
println("No diffs found in changelog")
71+
println("Changelog file is up-to-date")
7472

7573
object SnapshotDiffValidator:
76-
def summarize(previous: Dep, current: Dep): DiffSummary =
74+
def summarize(previous: Dep, current: Dep): DiffSummary =
7775
val versionDiff = Version.compareVersions(previous.version, current.version)
7876

7977
def traverseDep(oldDep: Option[Dep], newDep: Option[Dep], enteredFrom: Option[Dep] = None): List[Diff] =

0 commit comments

Comments
 (0)