From 6b4fa38aaae32cb0341fc0d7d1385b6c946d4588 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Piotr=20Krzemi=C5=84ski?= <git@krzeminski.it>
Date: Mon, 18 Jul 2022 14:49:44 +0200
Subject: [PATCH] [#13] feat!: use GitHub's way to print debug info

The only action's input - `verbose` - is now not needed.
---
 action-types.yml                              |  3 --
 action.yaml                                   |  5 ----
 .../githubactionstyping/ManifestsToReport.kt  | 30 ++++++++++---------
 .../github/ActionInputsReading.kt             | 11 -------
 4 files changed, 16 insertions(+), 33 deletions(-)
 delete mode 100644 src/main/kotlin/it/krzeminski/githubactionstyping/github/ActionInputsReading.kt

diff --git a/action-types.yml b/action-types.yml
index a601603..e69de29 100644
--- a/action-types.yml
+++ b/action-types.yml
@@ -1,3 +0,0 @@
-inputs:
-  verbose:
-    type: boolean
diff --git a/action.yaml b/action.yaml
index 920569b..bb9ae68 100644
--- a/action.yaml
+++ b/action.yaml
@@ -1,11 +1,6 @@
 name: GitHub Actions Typing
 description: Bring type-safety to your GitHub actions' API!
 author: Piotr Krzemiński
-inputs:
-  verbose:
-    description: 'Set to true to display debug information helpful when troubleshooting issues with this action.'
-    required: false
-    default: 'false'
 runs:
   using: 'docker'
   image: 'Dockerfile'
diff --git a/src/main/kotlin/it/krzeminski/githubactionstyping/ManifestsToReport.kt b/src/main/kotlin/it/krzeminski/githubactionstyping/ManifestsToReport.kt
index a65bc30..cff3ac4 100644
--- a/src/main/kotlin/it/krzeminski/githubactionstyping/ManifestsToReport.kt
+++ b/src/main/kotlin/it/krzeminski/githubactionstyping/ManifestsToReport.kt
@@ -1,6 +1,5 @@
 package it.krzeminski.githubactionstyping
 
-import it.krzeminski.githubactionstyping.github.getBooleanInput
 import it.krzeminski.githubactionstyping.parsing.TypesManifest
 import it.krzeminski.githubactionstyping.parsing.parseManifest
 import it.krzeminski.githubactionstyping.parsing.parseTypesManifest
@@ -30,23 +29,21 @@ fun manifestsToReport(manifest: String, typesManifest: String): Pair<Boolean, St
         throw IllegalStateException("The same set of outputs should exist in action manifest and types manifest!")
     }
 
-    if (getBooleanInput("verbose")) {
-        println("Action's manifest:")
-        println(manifest)
+    printlnDebug("Action's manifest:")
+    printlnDebug(manifest)
 
-        println("Parsed manifest:")
-        println(parsedManifest)
+    printlnDebug("Parsed manifest:")
+    printlnDebug(parsedManifest)
 
-        println("Action's types manifest:")
-        println(typesManifest)
+    printlnDebug("Action's types manifest:")
+    printlnDebug(typesManifest)
 
-        println("Parsed types manifest:")
-        println(parsedTypesManifest)
+    printlnDebug("Parsed types manifest:")
+    printlnDebug(parsedTypesManifest)
 
-        println()
-        println("==============================================")
-        println()
-    }
+    printlnDebug()
+    printlnDebug("==============================================")
+    printlnDebug()
 
     val validationResult = parsedTypesManifest.validate()
     val isValid = validationResult.overallResult is ItemValidationResult.Valid
@@ -54,3 +51,8 @@ fun manifestsToReport(manifest: String, typesManifest: String): Pair<Boolean, St
 
     return Pair(isValid, report)
 }
+
+private fun printlnDebug(string: Any? = null) =
+    (string ?: "").toString().lines()
+        .map { "::debug::$it" }
+        .forEach { println(it) }
diff --git a/src/main/kotlin/it/krzeminski/githubactionstyping/github/ActionInputsReading.kt b/src/main/kotlin/it/krzeminski/githubactionstyping/github/ActionInputsReading.kt
deleted file mode 100644
index 597cd1b..0000000
--- a/src/main/kotlin/it/krzeminski/githubactionstyping/github/ActionInputsReading.kt
+++ /dev/null
@@ -1,11 +0,0 @@
-package it.krzeminski.githubactionstyping.github
-
-import java.util.*
-
-fun getStringInput(inputName: String): String? {
-    val envVarName = "INPUT_${inputName.uppercase(Locale.getDefault())}"
-    return System.getenv(envVarName)
-}
-
-fun getBooleanInput(inputName: String): Boolean =
-    getStringInput(inputName)?.toBoolean() ?: false