1.7.1 for windows and some small cleanup (#17)

* 171 for windows and some small cleanup

* startWith -> startsWith

.github/workflows/tests.yaml

@@ -24,7 +24,6 @@ jobs:
       matrix:
         image:
           - "ubuntu-latest"
-          - "ubuntu-20.04"
           - "ubuntu-22.04"
           - "ubuntu-24.04"
           - "macos-latest"
@@ -54,19 +53,6 @@ jobs:
             version: '1.5'
             force: false
 
-          - image: "ubuntu-20.04"
-            version: '1.6'
-            force: true
-          - image: "ubuntu-20.04"
-            version: '1.6'
-            force: false
-          - image: "ubuntu-20.04"
-            version: '1.5'
-            force: true
-          - image: "ubuntu-20.04"
-            version: '1.5'
-            force: false
-
           - image: "ubuntu-22.04"
             version: '1.6'
             force: true
@@ -169,15 +155,13 @@ jobs:
         if: runner.os == 'Linux' || runner.os == 'macOS'
         # language=sh
         run: |
-          _installed='${{ steps.install-jq.outputs.installed }}'
-          _err=
           if [[ '${{ matrix.force }}' == 'true' ]]; then
             # enabling "force" must result in an install
             if [[ '${{ steps.install-jq.outputs.installed }}' != 'true' ]]; then
               echo 'Unexpected value for "installed":'
               echo 'Expected:   "true"'
               echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-              _err=1
+              exit 1
             fi
           else
             if [[ '${{ steps.install-jq.outputs.found }}' == 'true' ]]; then
@@ -186,7 +170,7 @@ jobs:
                 echo 'Unexpected value for "installed":'
                 echo 'Expected:   "false"'
                 echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-                _err=1
+                exit 1
               fi
             else
               # if not found, must be installed
@@ -194,12 +178,10 @@ jobs:
                 echo 'Unexpected value for "installed":'
                 echo 'Expected:   "true"'
                 echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-                _err=1
+                exit 1
               fi
             fi
           fi
-          if [ -n "${_err}" ]; then exit 1; fi;
-          
 
       - name: Check jq - Windows-ish
         if: runner.os == 'Windows' && matrix.force == 'true'
@@ -224,7 +206,6 @@ jobs:
           $ErrorActionPreference = 'Stop'
           Set-StrictMode -Version Latest
           $_installed='${{ steps.install-jq.outputs.installed }}'
-          $_err = 0
           if ("${{ matrix.force }}" -eq "true")
           {
               # enabling "force" must result in an install
@@ -233,7 +214,7 @@ jobs:
                   Write-Host "Unexpected value for installed"
                   Write-Host "Expected:   true"
                   Write-Host "Actual:     ${{ steps.install-jq.outputs.installed }}"
-                  $_err=1
+                  exit 1
               }
           }
           else
@@ -246,7 +227,7 @@ jobs:
                       Write-Host "Unexpected value for installed"
                       Write-Host "Expected:  false"
                       Write-Host "Actual:    ${{ steps.install-jq.outputs.installed }}"
-                      $_err=1
+                      exit 1
                   }
               }
               else
@@ -256,21 +237,16 @@ jobs:
                       Write-Host "Unexpected value for installed"
                       Write-Host "Expected:  true"
                       Write-Host "Actual:    ${{ steps.install-jq.outputs.installed }}"
-                      $_err=1
+                      exit 1
                   }
               }
           }
-          if ("${_err}" -ne 0)
-          {
-              exit 1
-          }
   test-container:
     strategy:
       matrix:
         image:
           - "ubuntu-24.04"
           - "ubuntu-22.04"
-          - "ubuntu-20.04"
         force:
           - 'true'
           - 'false'
@@ -296,35 +272,31 @@ jobs:
       - name: Check jq
         # language=sh
         run: |
-          _err=
           _which="$(which jq)"
           _vers="$(jq --version)"
           if [[ "${_which}" != "$RUNNER_TOOL_CACHE/jq/jq" ]]; then
             echo "jq found at unexpected path."
             echo "  Expected:   \"$RUNNER_TOOL_CACHE/jq/jq\""
             echo "  Actual:     \"${_which}\""
-            _err=1
+            exit 1
           fi
           if [[ "${_vers}" != 'jq-${{ matrix.version }}' ]]; then
             echo "jq --version returned unexpected value"
             echo '  Expected:   "jq-${{ matrix.version }}"'
             echo "  Actual:     \"${_vers}\""
-            _err=1
+            exit 1
           fi
-          if [ -n "${_err}" ]; then exit 1; fi;
 
       - name: Check Outputs
         # language=sh
         run: |
-          _installed='${{ steps.install-jq.outputs.installed }}'
-          _err=
           if [[ '${{ matrix.force }}' == 'true' ]]; then
             # enabling "force" must result in an install
             if [[ '${{ steps.install-jq.outputs.installed }}' != 'true' ]]; then
               echo 'Unexpected value for "installed":'
               echo 'Expected:   "true"'
               echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-              _err=1
+              exit 1
             fi
           else
             if [[ '${{ steps.install-jq.outputs.found }}' == 'true' ]]; then
@@ -333,7 +305,7 @@ jobs:
                 echo 'Unexpected value for "installed":'
                 echo 'Expected:   "false"'
                 echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-                _err=1
+                exit 1
               fi
             else
               # if not found, must be installed
@@ -341,8 +313,7 @@ jobs:
                 echo 'Unexpected value for "installed":'
                 echo 'Expected:   "true"'
                 echo 'Actual:     "${{ steps.install-jq.outputs.installed }}"'
-                _err=1
+                exit 1
               fi
             fi
           fi
-          if [ -n "${_err}" ]; then exit 1; fi;

README.md

@@ -5,6 +5,7 @@ Multiplatform [jq](https://github.com/stedolan/jq) installer action
 
 This action is tested against the following versions of JQ:
 
+- [1.7.1](https://github.com/jqlang/jq/releases/tag/jq-1.7.1)
 - [1.7](https://github.com/jqlang/jq/releases/tag/jq-1.7)
 - [1.6](https://github.com/jqlang/jq/releases/tag/jq-1.6)
 - [1.5](https://github.com/jqlang/jq/releases/tag/jq-1.5)
@@ -29,7 +30,7 @@ This action is tested against the following versions of JQ:
   version:
     required: false
     description: "Version of jq to install"
-    default: "1.7"
+    default: "1.7.1"
 ```
 
 This must be a version with a [corresponding release](https://github.com/stedolan/jq/releases).

action.yaml

@@ -72,14 +72,14 @@ runs:
         }
 
     - name: 'Install jq - Windows-ish non-1.7'
-      if: runner.os == 'Windows' && (inputs.version == '1.5' || inputs.version == '1.6') && (steps.jq-check-windows.outputs.found == 'false' || inputs.force == 'true')
+      if: runner.os == 'Windows' && (startsWith(inputs.version, '1.5') || startsWith(inputs.version, '1.6')) && (steps.jq-check-windows.outputs.found == 'false' || inputs.force == 'true')
       shell: powershell
       env:
         JQ_VERSION: '${{ inputs.version }}'
       run: ${{ github.action_path }}\scripts\windowsish.ps1
 
     - name: 'Install jq - Windows-ish 1.7'
-      if: runner.os == 'Windows' && inputs.version == '1.7' && (steps.jq-check-windows.outputs.found == 'false' || inputs.force == 'true')
+      if: runner.os == 'Windows' && startsWith(inputs.version, '1.7') && (steps.jq-check-windows.outputs.found == 'false' || inputs.force == 'true')
       shell: powershell
       env:
         JQ_VERSION: '${{ inputs.version }}'