Merge pull request #30 from actions/decouple-telemetry

Remove the support for older dynamic jekyll workflows
actions · Apr 15, 2022 · a7af90c · a7af90c
2 parents eaf36f4 + 62c650c
commit a7af90c
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 9 deletions.
diff --git a/action.yml b/action.yml
@@ -7,6 +7,7 @@ inputs:
   emit_telemetry:
     description: 'Should this action only emit build telemetry instead of deploying the build artifact?'
     required: false
+    default: "false"
   token:
     description: 'GitHub token'
     default: ${{ github.token }}

diff --git a/dist/index.js b/dist/index.js
diff --git a/dist/index.js.map b/dist/index.js.map
diff --git a/src/index.js b/src/index.js
@@ -55,9 +55,6 @@ process.on('SIGTERM', cancelHandler)
 const emitTelemetry = core.getInput("emit_telemetry")
 if (emitTelemetry === "true") {
   require('./pre')
-} else if (emitTelemetry === "false") {
-  main()
 } else {
-  // If emit_telemetry is not set, that indicates an older version of the dynamic workflow that doesn't separate telemetry from deployment
-  main().then(() => require('./pre'))
+  main()
 }