Update README.md

Reviewed-on: #2

.gitea/workflows/test.yaml

@@ -1,14 +1,10 @@
 name: "test"
 run-name: "test"
-on: 
+on: push
-  push:
-    tags:
-      - "v*"
-      - "!v*-beta*"
 
 jobs:
   test:
-    runs-on: [smoketest]
+    runs-on: action
     steps:
       - uses: actions/checkout@v4
 
@@ -20,10 +16,11 @@ jobs:
       - name: "run go install script"
         shell: bash
         env:
-          GO_INSTALL_COMMANDS: github.com/jake-young-dev/kelp@v0.0.9
+          GO_INSTALL_COMMANDS: code.jakeyoungdev.com/jake/mctl@v0.3.4
+          GO_INSTALL_ARCH: amd64
         run: |
           chmod +x install.sh
-          ./install.sh amd64 no
+          ./install.sh
 
       - name: "check go version"
         shell: bash
@@ -31,4 +28,4 @@ jobs:
 
       - name: "check go installed command"
         shell: bash
-        run: kelp -v | grep "kelp version v0.0.9"
+        run: mctl -v | grep "mctl version v0.3.4"

README.md

@@ -1,4 +1,5 @@
 # install-go
+![tests](https://code.jakeyoungdev.com/actions/install-go/actions/workflows/test.yaml/badge.svg?branch=main&event=push) <br />
 An extremely simple github action to install golang and golang commands (using "go install") on linux hosts. This provides the ability to install and setup golang without node/typescript which is required when using [actions/setup-go](https://github.com/actions/setup-go). 
 
 
@@ -9,7 +10,7 @@ Use a tagged release to avoid unexpected changes that may come to the master bra
     uses: https://code.jakeyoungdev.com/actions/install-go@master
     with:
         commands: |
-            github.com/jake-young-dev/kelp@v0.0.9
+            code.jakeyoungdev.com/jake/mctl@v0.3.3
 ```
 
 #### Go verions

TODO.txt

@@ -1,3 +0,0 @@
-Comments are left in code using TODO# to track possible implementation location.
-
-TODO1. check current installed version to see if it matches requested install version, only error if they don't match

install.sh

@@ -1,5 +1,24 @@
 #!/bin/bash
 
+#installs any commands set in the command input value
+function installCommands {
+  #if commands input is set pull the input and install them all using 'go install'
+  if [[ "$GO_INSTALL_COMMANDS" != "." ]]; then # "." is used as a default
+    #allows for multiple cmds using |
+    INPUT_ARR=( $GO_INSTALL_COMMANDS )
+    for i in "${INPUT_ARR[@]}"; do
+        echo "Installing go command from ${i}"
+        if go install ${i}; then
+          echo "Command setup for ${i}"
+        else
+          echo "FATAL: Unable to install ${i} with go"
+          exit 1
+        fi
+    done
+  fi
+}
+
+#start script
 echo "Starting Golang install"
 
 #validate version and architecture
@@ -30,17 +49,24 @@ echo "Checking if go is already installed"
 # -v writes string that indicates command or command path to output, prevents command not found error
 GO_CHECK=$(command -v go)
 if [[ "$GO_CHECK" ]]; then
-  #if purge flag is set remove old go files
+  #if the version of go matches the requested version, skip
-  if [[ "$GO_INSTALL_PURGE" == "yes" ]]; then
+  GVC=$(go version)
+  if [[ "$GVC" == "go version go${DL_VERSION} linux/${DL_ARCH}" ]]; then
+    echo "Go ${DL_VERSION} already installed, skipping!"
+    installCommands
+    exit 0
+  fi
+  #if the purge flag is not set and version does not match, exit
+  if [[ "$GO_INSTALL_PURGE" != "yes" ]]; then
+    echo "FATAL: The wrong version of Go is already installed, set purge to 'yes' if you wish to update installed version"
+    exit 1
+  else
     echo "Removing old go versions"
     sudo rm -r /usr/bin/go 
     sudo rm -r /usr/bin/gofmt
-  else
-    #TODO1: if installed version matches requested version we can consider the job successful
-    echo "FATAL: Go is already installed, set purge to 'yes' if you wish to update installed version"
-    exit 1
   fi
 fi
+
 echo "Ready for install"
 
 echo "Downloading go files for ${DL_VERSION}/${DL_ARCH}"
@@ -82,17 +108,4 @@ GP=$(go env GOPATH)/bin
 export PATH=$PATH:$GP
 echo "$GP" >> "$GITHUB_PATH"
 
-#if commands input is set pull the input and install them all using 'go install'
+installCommands
-if [[ "$GO_INSTALL_COMMANDS" != "." ]]; then # "." is used as a default
-  #allows for multiple cmds using |
-  INPUT_ARR=( $GO_INSTALL_COMMANDS )
-  for i in "${INPUT_ARR[@]}"; do
-      echo "Installing go command from ${i}"
-      if go install ${i}; then
-        echo "Command setup for ${i}"
-      else
-        echo "FATAL: Unable to install ${i} with go"
-        exit 1
-      fi
-  done
-fi