[STYLE] Minor fixes and tweaks (- WIP PR #227 -)

Changes in file .github/workflows/makefile-lint.yml:
 - fixed line numebrs in diognostics

Changes in file .github/workflows/markdown-lint.yml:
 - fixed a typo
 - fixed line numbers in diognostics

Author: reactive-firewall

.github/workflows/makefile-lint.yml

@@ -56,15 +56,15 @@ jobs:
           YAML_ARGS="${{ vars.YAML_ARGS }}"
           if [[ "${YAML_ARGS}" =~ ^*[!a-zA-Z0-9\ \-\.]*$ ]]; then
               ERR_MSG_STUB="Invalid YAML_ARGS: contains unauthorized characters!"
-              ERR_LOC="file=.github/workflows/markdown-lint.yml,line=37,endLine=42,"
+              ERR_LOC="file=.github/workflows/makefile-lint.yml,line=56,endLine=57,"
               printf "::error %s%s\n" "${ERR_LOC}" "title=VALIDATION_ERROR::ERROR ${ERR_MSG_STUB}"
               exit 1
           else
-              printf "::notice file=.github/workflows/markdown-lint.yml,%s %s\n" \
-                "line=36,endLine=47,title=Validation::Validating workflow with args:" \
+              printf "::notice file=.github/workflows/makefile-lint.yml,%s %s\n" \
+                "line=67,endLine=67,title=Validation::Validating workflow with args:" \
                 "${YAML_ARGS}"
               read -ra ARGS <<< "${YAML_ARGS}"
-              yamllint "${ARGS[@]}" .github/workflows/markdown-lint.yml
+              yamllint "${ARGS[@]}" .github/workflows/makefile-lint.yml
           fi
       - name: Install checkmake
         if: ${{ success() }}
@@ -78,14 +78,14 @@ jobs:
           ACTUAL_SHA=$(git rev-parse HEAD)
           if [ "$EXPECTED_SHA" != "$ACTUAL_SHA" ]; then
             ERR_MSG_STUB="Checkmake repository hash verification failed."
-            ERR_DETAILS="file=.github/workflows/makefile-lint.yml,line=42,endLine=51,title=FAILURE"
+            ERR_DETAILS="file=.github/workflows/makefile-lint.yml,line=75,endLine=79,title=FAILURE"
             ERR_MSG_LINE="::error ${ERR_DETAILS}::ERROR ${ERR_MSG_STUB}"
             printf "%s\n" "${ERR_MSG_LINE}"
             exit 1;
           fi
           make && make install || { \
             ERR_MSG_STUB="Checkmake build failed." \
-            ERR_DETS="file=.github/workflows/makefile-lint.yml,line=52,endLine=59,title=FAILURE" \
+            ERR_DETS="file=.github/workflows/makefile-lint.yml,line=86,endLine=92,title=FAILURE" \
             ERR_MSG_LINE="::error ${ERR_DETS}::ERROR ${ERR_MSG_STUB}" \
             printf "%s\n" "${ERR_MSG_LINE}" \
             exit 1 \
@@ -120,10 +120,10 @@ jobs:
               "${file_stub}" "${line_start}" "${line_end}" "${msg}"
           }
           if [ ! -x "$TOOL_PATH" ]; then
-            { print_error "$TOOL_PATH not found or not executable." 101 103; exit 1; } ;
+            { print_error "$TOOL_PATH not found or not executable." 122 124; exit 1; } ;
           fi
           FILE="${{ steps.makefiles.outputs.files }}" ;
           printf "::group::%s\n" "${FILE}" ;
-          "$TOOL_PATH" "${FILE}" || { print_error "Linting failed." 106 106; exit 1; } ;
+          "$TOOL_PATH" "${FILE}" || { print_error "Linting failed." 127 127; exit 1; } ;
           printf "::endgroup::\n" ; unset FILE 2>/dev/null || true ;
         if: ${{ !cancelled() && steps.makefiles.outputs.files != '' }}

.github/workflows/markdown-lint.yml

@@ -42,22 +42,30 @@ jobs:
           YAML_ARGS="${{ vars.YAML_ARGS }}"
           if [[ "${YAML_ARGS}" =~ ^*[!a-zA-Z0-9\ \-\.]*$ ]]; then
               ERR_MSG_STUB="Invalid YAML_ARGS: contains unauthorized characters!"
-              ERR_LOC="file=.github/workflows/markdown-lint.yml,line=37,endLine=42,"
+              ERR_LOC="file=.github/workflows/markdown-lint.yml,line=42,endLine=43,"
               printf "::error %s%s\n" "${ERR_LOC}" "title=VALIDATION_ERROR::ERROR ${ERR_MSG_STUB}"
               exit 1
           else
               printf "::notice file=.github/workflows/markdown-lint.yml,%s %s\n" \
-                "line=42,endLine=43,title=Validation::Validating workflow with args:" \
+                "line=53,endLine=53,title=Validation::Validating workflow with args:" \
                 "${YAML_ARGS}"
               read -ra ARGS <<< "${YAML_ARGS}"
               yamllint "${ARGS[@]}" .github/workflows/markdown-lint.yml
           fi
 
+      - name: Lint YAML config for markdown
+        if: ${{ success() }}
+        run: |
+          YAML_ARGS="${{ vars.YAML_ARGS }}"
+          read -ra ARGS <<< "${YAML_ARGS}"
+          yamllint "${ARGS[@]}" .markdownlint.yaml
+
       - name: Install NPM Dependencies
+        if: ${{ success() }}
         run: |
           # npm install
           ERR_MSG="NPM package installation failed"
-          ERR_LOC_1="file=.github/workflows/markdown-lint.yml,line=62,endLine=62"
+          ERR_LOC_1="file=.github/workflows/markdown-lint.yml,line=70,endLine=70"
           ERR_LOC_2="file=package.json,line=2,endLine=4"
           if ! NPM_ERROR=$(npm install 2>&1); then
           for LOC in "${ERR_LOC_1}" "${ERR_LOC_2}"; do
@@ -66,14 +74,8 @@ jobs:
           done
           fi
 
-      - name: Lint YAML config for markdown
-        run: |
-          YAML_ARGS="${{ vars.YAML_ARGS }}"
-          read -ra ARGS <<< "${YAML_ARGS}"
-          yamllint ${{ vars.YAML_ARGS }} .markdownlint.yaml
-
       - name: Lint Markdown Files with YAML config
-        if: ${{ !cancelled() }}
+        if: ${{ success() }}
         run: |
           git ls-files --exclude-standard -z -- *.md **/*.md |\
           xargs -0 -I{} npm run lint:md -- "{}";