Skip to content

Commit

Permalink
Remove the support for older dynamic jekyll workflows
Browse files Browse the repository at this point in the history
The pages dynamic workflow now decouples telemetry from deploy for 100%
of users, so we can remove this old conditional.
  • Loading branch information
TooManyBees committed Apr 4, 2022
1 parent eaf36f4 commit 62c650c
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 9 deletions.
1 change: 1 addition & 0 deletions action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -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 }}
Expand Down
5 changes: 1 addition & 4 deletions dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion dist/index.js.map

Large diffs are not rendered by default.

5 changes: 1 addition & 4 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -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()
}

0 comments on commit 62c650c

Please sign in to comment.