Skip to content

Commit ebe3d7a

Browse files
authored
ci(install-proof): bind KIN_NO_SETUP to sh, not curl (FIR-1012) (#44)
`KIN_NO_SETUP=1 curl … | sh` sets the var only for curl; the piped script never sees it and tries interactive setup against a non-controlling /dev/tty (ubuntu). Pipe to `KIN_NO_SETUP=1 sh` so the CI/non-interactive path skips setup. (The same correction is needed in the documented command on the kinlab setup page / README.) Signed-off-by: Troy Fortin <troy@firelock.io>
1 parent c565ca3 commit ebe3d7a

1 file changed

Lines changed: 5 additions & 2 deletions

File tree

.github/workflows/install-proof.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,11 @@ jobs:
3030
if: runner.os != 'Windows'
3131
run: |
3232
set -eu
33-
# The exact command from the README / kinlab.ai setup page.
34-
KIN_NO_SETUP=1 curl -fsSL https://get.kinlab.dev/install | sh
33+
# CI/non-interactive install. NOTE the env var binds to `sh`, not
34+
# `curl` — `KIN_NO_SETUP=1 curl … | sh` would set it only for curl and
35+
# the piped script would never see it (it would then try the
36+
# interactive setup against a non-controlling /dev/tty).
37+
curl -fsSL https://get.kinlab.dev/install | KIN_NO_SETUP=1 sh
3538
# install.sh installs to ~/.kin/bin and verifies the per-artifact sha256.
3639
echo "$HOME/.kin/bin" >> "$GITHUB_PATH"
3740

0 commit comments

Comments
 (0)