Merge pull request #83 from gvatsal60/fix/minor_fixes_Vg

Minor Fixes

Author: andmpel

.devcontainer/devcontainer.json

@@ -3,11 +3,12 @@
     "vscode": {
       "extensions": [
         "DavidAnson.vscode-markdownlint",
-        "MindpathTechnologyLimited.code-error-lens",
         "Tyriar.sort-lines",
         "foxundermoon.shell-format",
         "mads-hartmann.bash-ide-vscode",
-        "streetsidesoftware.code-spell-checker"
+        "streetsidesoftware.code-spell-checker",
+        "timonwong.shellcheck",
+        "usernamehw.errorlens"
       ],
       "settings": {
         "[shellscript]": {
@@ -33,7 +34,7 @@
       }
     }
   },
-  "image": "mcr.microsoft.com/devcontainers/base:noble",
+  "image": "mcr.microsoft.com/devcontainers/base:debian",
   "runArgs": [
     "--rm",
     "--name=vsc-${localEnv:USER}-${containerWorkspaceFolderBasename}-${devcontainerId}"

.github/workflows/pre-commit-pr.yaml

@@ -13,7 +13,7 @@ jobs:
         env:
           PR_URL: ${{github.event.pull_request.html_url}}
           GH_TOKEN: ${{secrets.GITHUB_TOKEN}}
-      - name: Enable auto-merge for Dependabot PRs
+      - name: Enable auto-merge for pre-commit PRs
         run: gh pr merge --auto --merge "$PR_URL"
         env:
           PR_URL: ${{github.event.pull_request.html_url}}

.markdownlint.json

@@ -0,0 +1,5 @@
+{
+  "MD013": {
+    "line_length": 999
+  }
+}

.markdownlint.yaml

@@ -68,11 +68,6 @@ repos:
     hooks:
       - id: talisman-commit
         entry: cmd --githook pre-commit
-  # Checkmake for linting Makefiles
-  - repo: https://github.com/mrtazz/checkmake
-    rev: "0.2.2"
-    hooks:
-      - id: checkmake
   # Markdownlint for linting Markdown files
   - repo: https://github.com/DavidAnson/markdownlint-cli2
     rev: v0.18.1

.pre-commit-config.yaml

@@ -73,7 +73,7 @@ update_gem() {
     GEM_PATH=$(which gem)
 
     # Check if the path does not match the expected path
-    if [ "$GEM_PATH" = "/usr/bin/gem" ]; then
+    if [ "${GEM_PATH}" = "/usr/bin/gem" ]; then
         print_err "gem is not installed."
         return
     fi