[airflow] branch master updated: Fixed shellcheck error with static checks (#15450)

[po...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/master by this push:
     new f97d702  Fixed shellcheck error with static checks (#15450)
f97d702 is described below

commit f97d7023bf79a2f9078aa7785572a381e4c64dd1
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Tue Apr 20 12:00:16 2021 +0200

    Fixed shellcheck error with static checks (#15450)
    
    Shellcheck released today broke static checks. This PR fixes it
    and pins shellcheck to specific version to avoid it in the future.
---
 .pre-commit-config.yaml | 2 +-
 dev/remove_artifacts.sh | 1 -
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
index 50e18ed..b48f1e7 100644
--- a/.pre-commit-config.yaml
+++ b/.pre-commit-config.yaml
@@ -428,7 +428,7 @@ repos:
       - id: shellcheck
         name: Check Shell scripts syntax correctness
         language: docker_image
-        entry: koalaman/shellcheck:stable -x -a
+        entry: koalaman/shellcheck:v0.7.2 -x -a
         files: ^breeze$|^breeze-complete$|\.sh$|^hooks/build$|^hooks/push$|\.bash$|\.bats$
       - id: bats-tests
         name: Run BATS bash tests for changed Breeze bash files
diff --git a/dev/remove_artifacts.sh b/dev/remove_artifacts.sh
index 49bc4c0..7b4ab99 100755
--- a/dev/remove_artifacts.sh
+++ b/dev/remove_artifacts.sh
@@ -40,7 +40,6 @@ if [[ -z ${GITHUB_TOKEN} ]]; then
     echo 2>&1
     exit 2
 fi
-GITHUB_TOKEN=${GITHUB_TOKEN}
 readonly GITHUB_TOKEN
 
 function github_api_call() {