Merge pull request Bash-it#1936 from NoahGorny/fix-shellcheck-warnings

plugins: Fix shellcheck warnings on already linted files
jmthvt · Sep 12, 2021 · 99eab7a · 99eab7a
2 parents f0690c0 + c6f6d8c
commit 99eab7a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/plugins/available/alias-completion.plugin.bash b/plugins/available/alias-completion.plugin.bash
@@ -70,7 +70,7 @@ function alias_completion {
 			local compl_func="${new_completion/#* -F /}"
 			compl_func="${compl_func%% *}"
 			# avoid recursive call loops by ignoring our own functions
-			if [[ "${compl_func#_$namespace::}" == "$compl_func" ]]; then
+			if [[ "${compl_func#_"$namespace"::}" == "$compl_func" ]]; then
 				local compl_wrapper="_${namespace}::${alias_name}"
 				echo "function $compl_wrapper {
                         local compl_word=\$2

diff --git a/plugins/available/git.plugin.bash b/plugins/available/git.plugin.bash
@@ -309,7 +309,7 @@ function git-changelog() {
 		# shellcheck disable=SC2162
 		git log "$1" --no-merges --format="%cd" --date=short | sort -u -r | while read DATE; do
 			echo
-			echo [$DATE]
+			echo "[$DATE]"
 			git log --no-merges --format=" * (%h) %s by %an <%ae>" --since="$DATE 00:00:00" --until="$DATE 24:00:00"
 			NEXT=$DATE
 		done