remove ignorance check for unset vars

.github/workflows/commitlint.yaml

@@ -37,4 +37,5 @@ jobs:
           pull_request_title: ${{ github.event.pull_request.title }}
         run: |
           echo "module.exports = {extends: ['@commitlint/config-conventional']}" > commitlint.config.js
+          # shellcheck disable=SC2154
           echo "$pull_request_title" | npx commitlint

tasks/lint.yaml

@@ -53,8 +53,7 @@ tasks:
           # -e SC2001 Ignore sed replace suggestions (${text//search/replacement} syntax doesn't support regex so this suggestion often won't work)
           # -e SC2002 Ignore rejection of `cat |` (redirection syntax is not always familiar to developers and while cat _could_ not exist that is extremely unlikely)
           # -e SC2181 Ignore rejection of $? ($? can be confusing in some circumstances but also more readable in others with more complex commands)
-          # -e SC2154 Ignore unset variables (in many cases env vars will come from the environment and we should assume they are set)
-          export SHELLCHECK_OPTS="-x -e SC2016 -e SC2050 -e SC2001 -e SC2002 -e SC2181 -e SC2154"
+          export SHELLCHECK_OPTS="-x -e SC2016 -e SC2050 -e SC2001 -e SC2002 -e SC2181"
 
           # Function to lint scripts inside `---` delimited files
           lint_scripts() {