Merge pull request #60 from Tumiya/feature/cleanup-unnecessary-shellcheck-disabled-issues

Clean up unnecessary Shellcheck disabled issues

Author: Azbagheri

gitrise.sh

@@ -1,4 +1,5 @@
 #!/usr/bin/env bash
+
 # shellcheck disable=SC2155
 # disbales "Declare and assign separately to avoid masking return values."
 # shellcheck disable=SC2120

tests/build_artifacts_tests.sh

@@ -1,5 +1,7 @@
 #!/usr/bin/env bash
 
+#shellcheck disable=SC1091
+
 source ./gitrise.sh -T
 
 testFetchingBuildArtifactsSlugs() {

tests/build_info_tests.sh

@@ -1,10 +1,6 @@
 #!/usr/bin/env bash
 
-# shellcheck disable=SC1091,SC2155,SC2154,SC2034
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
-# var is referenced but not assigned.
-# var appears unused
+# shellcheck disable=SC1091,SC2155
 
 source ./gitrise.sh -T
 

tests/build_logs_tests.sh

@@ -1,9 +1,6 @@
 #!/usr/bin/env bash
 
-# shellcheck disable=SC1091,SC2155,SC2154
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
-# var is referenced but not assigned.
+# shellcheck disable=SC1091,SC2155
 
 source ./gitrise.sh -T
 

tests/build_status_message_tests.sh

@@ -1,8 +1,7 @@
 #!/usr/bin/env bash
 
 # shellcheck disable=SC1091,SC2155
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
+
 source ./gitrise.sh -T
 
 testBuildTimedOutMessage() {

tests/download_single_artifacts_tests.sh

@@ -1,10 +1,6 @@
 #!/usr/bin/env bash
 
-# shellcheck disable=SC1091,SC2155,SC2154,SC2034
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
-# var is referenced but not assigned.
-# var appears unused
+# shellcheck disable=SC1091,SC2155
 
 source ./gitrise.sh -T
 

tests/gitrise_logs_tests.sh

@@ -1,7 +1,6 @@
 #!/usr/bin/env bash
 
-# shellcheck disable=SC1091,SC2155,SC2154
-# Not following: (error message here)
+# shellcheck disable=SC1091,SC2155
 
 source ./gitrise.sh -T -d
 

tests/input_validation_tests.sh

@@ -1,8 +1,6 @@
 #!/usr/bin/env bash
 
 # shellcheck disable=SC1091,SC2155
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
 
 testMissingWorkflow() {
     local expected_message="ERROR: Missing arguments(s). All these args must be passed: --workflow,--slug,--access-token"

tests/options_tests.sh

@@ -1,8 +1,7 @@
 #!/usr/bin/env bash
 
 # shellcheck disable=SC1091,SC2155
-# Not following: (error message here)
-# Declare and assign separately to avoid masking return values.
+
 source ./gitrise.sh -T
 
 testEnvVars() {