From 8a8da6a7f530e2e2ddd8527e92070b8d75cdf317 Mon Sep 17 00:00:00 2001 From: plocket <52798256+plocket@users.noreply.github.com> Date: Mon, 4 Dec 2023 12:39:47 -0500 Subject: [PATCH 1/4] Troubleshoot workflow input `enable_tmate` Try the solution here: https://stackoverflow.com/a/70077545/14144258 to use `fromJSON` and echo that value --- .github/workflows/github_server.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/github_server.yml b/.github/workflows/github_server.yml index f6b835ed..86013868 100644 --- a/.github/workflows/github_server.yml +++ b/.github/workflows/github_server.yml @@ -40,6 +40,9 @@ jobs: # relative paths to action files - uses: actions/checkout@v3 + - run: echo ${{ fromJSON(github.event.inputs.enable_tmate) }} + shell: bash + - name: ALKiln - Start the isolated temporary docassemble server on GitHub id: github_server #### Developer note: you'll need to replace `.` with the path to @@ -57,7 +60,7 @@ jobs: #### Developer note: You probably don't need this # Optional debugging to explore things like config issues - name: Docker debug tmate session - if: ${{ github.event.inputs.enable_tmate }} + if: fromJSON(github.event.inputs.enable_tmate) uses: mxschmitt/action-tmate@v3 #### Developer note: You probably don't need this From dc93d9824e222fe10e8f7f99fab723305574bfb0 Mon Sep 17 00:00:00 2001 From: plocket <52798256+plocket@users.noreply.github.com> Date: Mon, 4 Dec 2023 12:48:26 -0500 Subject: [PATCH 2/4] Use inputs.var as opposed to github.event.input.var --- .github/workflows/github_server.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/github_server.yml b/.github/workflows/github_server.yml index 86013868..4d116f6f 100644 --- a/.github/workflows/github_server.yml +++ b/.github/workflows/github_server.yml @@ -40,7 +40,7 @@ jobs: # relative paths to action files - uses: actions/checkout@v3 - - run: echo ${{ fromJSON(github.event.inputs.enable_tmate) }} + - run: echo ${{ fromJSON(inputs.enable_tmate) }} shell: bash - name: ALKiln - Start the isolated temporary docassemble server on GitHub @@ -60,7 +60,7 @@ jobs: #### Developer note: You probably don't need this # Optional debugging to explore things like config issues - name: Docker debug tmate session - if: fromJSON(github.event.inputs.enable_tmate) + if: fromJSON(inputs.enable_tmate) uses: mxschmitt/action-tmate@v3 #### Developer note: You probably don't need this From f7830997403970f66b710dd1af7a5f4a23f63d73 Mon Sep 17 00:00:00 2001 From: plocket <52798256+plocket@users.noreply.github.com> Date: Mon, 4 Dec 2023 12:51:08 -0500 Subject: [PATCH 3/4] Use toJson() instead of fromJson() --- .github/workflows/github_server.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/github_server.yml b/.github/workflows/github_server.yml index 4d116f6f..f57cbfba 100644 --- a/.github/workflows/github_server.yml +++ b/.github/workflows/github_server.yml @@ -40,7 +40,7 @@ jobs: # relative paths to action files - uses: actions/checkout@v3 - - run: echo ${{ fromJSON(inputs.enable_tmate) }} + - run: echo ${{ toJSON(inputs.enable_tmate) }} shell: bash - name: ALKiln - Start the isolated temporary docassemble server on GitHub From 4756c325238b586e3fc63974d02ccabc0fb6686b Mon Sep 17 00:00:00 2001 From: plocket <52798256+plocket@users.noreply.github.com> Date: Mon, 4 Dec 2023 14:24:17 -0500 Subject: [PATCH 4/4] Compare to 'true', I suppose, since boolean didn't work before --- .github/workflows/github_server.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/github_server.yml b/.github/workflows/github_server.yml index f57cbfba..d4ffb545 100644 --- a/.github/workflows/github_server.yml +++ b/.github/workflows/github_server.yml @@ -60,7 +60,7 @@ jobs: #### Developer note: You probably don't need this # Optional debugging to explore things like config issues - name: Docker debug tmate session - if: fromJSON(inputs.enable_tmate) + if: ${{ toJSON(inputs.enable_tmate) == 'true' }} uses: mxschmitt/action-tmate@v3 #### Developer note: You probably don't need this