From a127408e938ecd7060d7f81dec557bc5eaf06d75 Mon Sep 17 00:00:00 2001 From: Ian Smith Date: Fri, 14 May 2021 00:31:15 -0700 Subject: run shellcheck (#97) * run shellcheck * Update .github/workflows/shellcheck.yml Co-authored-by: Conrad Ludgate * shellcheck fixes in which i am highly confident * shellcheck fixes in which I am less confident - not executing this is fine, right? * SC2155 In src/shell/atuin.bash line 1: export ATUIN_SESSION=$(atuin uuid) ^-----------^ SC2155: Declare and assign separately to avoid masking return values. * shellcheck doesn't support zsh, per https://github.com/koalaman/shellcheck/wiki/SC1071 * yaml fix * gotta checkout our code, too * yaml fix * action spelling * exclude .zsh * Shellcheck doesn't support zsh, per https://github.com/koalaman/shellcheck/wiki/SC1071, and the ignore: param in ludeeus/action-shellcheck only supports _directories_, not _files_. So instead, we manually add any error the shellcheck step finds in the file to the above line ... * comment all the ignores * Update src/shell/atuin.bash Co-authored-by: Ellie Huxtable * new zsh plugin shellcheck errors * new zsh plugin shellcheck errors, pt 2 Co-authored-by: Conrad Ludgate Co-authored-by: Ellie Huxtable --- src/shell/atuin.bash | 3 ++- src/shell/atuin.zsh | 8 ++++++++ 2 files changed, 10 insertions(+), 1 deletion(-) (limited to 'src') diff --git a/src/shell/atuin.bash b/src/shell/atuin.bash index 038e4423..40bed30e 100644 --- a/src/shell/atuin.bash +++ b/src/shell/atuin.bash @@ -1,4 +1,5 @@ -export ATUIN_SESSION=$(atuin uuid) +ATUIN_SESSION=$(atuin uuid) +export ATUIN_SESSION _atuin_preexec() { id=$(atuin history start "$1") diff --git a/src/shell/atuin.zsh b/src/shell/atuin.zsh index 15f2492b..32cbe6a1 100644 --- a/src/shell/atuin.zsh +++ b/src/shell/atuin.zsh @@ -1,3 +1,11 @@ +# shellcheck disable=SC2034,SC2153,SC2086,SC2155 + +# Above line is because shellcheck doesn't support zsh, per +# https://github.com/koalaman/shellcheck/wiki/SC1071, and the ignore: param in +# ludeeus/action-shellcheck only supports _directories_, not _files_. So +# instead, we manually add any error the shellcheck step finds in the file to +# the above line ... + # Source this in your ~/.zshrc autoload -U add-zsh-hook -- cgit v1.2.3