Improve CI workflow and Go version compatibility

Author: BronzonTech-Cloud

.github/workflows/ci.yml

@@ -28,10 +28,7 @@ jobs:
           go-version: 1.19
           cache: true  # Enable cache now that we have a go.sum file
 
-      # Skip go mod download since we don't have external dependencies yet
-      # If we add dependencies later, we can uncomment this
-      # - name: Download dependencies
-      #   run: go mod download
+      # We don't need go mod download since we don't have external dependencies
 
       - name: Build
         run: go build -v ./cmd/stremax
@@ -47,7 +44,6 @@ jobs:
     name: Test
     runs-on: ubuntu-latest
     timeout-minutes: 10
-    needs: build
 
     steps:
       - name: Checkout code
@@ -146,7 +142,6 @@ jobs:
     name: Benchmarks
     runs-on: ubuntu-latest
     timeout-minutes: 5
-    needs: test
 
     steps:
       - name: Checkout code

.github/workflows/go-compatibility.yml

@@ -0,0 +1,34 @@
+name: Go Compatibility
+
+on:
+  workflow_dispatch:  # Allow manual triggering
+  schedule:
+    - cron: '0 0 * * 0'  # Run weekly on Sundays at midnight UTC
+
+jobs:
+  test-compatibility:
+    name: Test Go Compatibility
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go-version: [1.16, 1.17, 1.18, 1.19, 1.20, 1.21]
+      fail-fast: false  # Continue with other tests even if one fails
+    
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+
+      - name: Set up Go ${{ matrix.go-version }}
+        uses: actions/setup-go@v3
+        with:
+          go-version: ${{ matrix.go-version }}
+          cache: false  # Disable cache for compatibility testing
+
+      - name: Display Go version
+        run: go version
+
+      - name: Build
+        run: go build -v ./cmd/stremax || echo "Build failed with Go ${{ matrix.go-version }}"
+
+      - name: Run tests
+        run: go test ./... || echo "Tests failed with Go ${{ matrix.go-version }}" 

run_tests.sh

@@ -13,36 +13,52 @@ echo "=================================="
 GO_VERSION=$(go version | awk '{print $3}' | sed 's/go//')
 echo -e "${YELLOW}Using Go version: ${GO_VERSION}${NC}"
 
-# Only run go mod download if we're using Go 1.11 or higher
-if [ "$(printf '%s\n' "1.11" "$GO_VERSION" | sort -V | head -n1)" = "1.11" ]; then
-  echo -e "${YELLOW}Go modules supported. Checking dependencies...${NC}"
-  # Skip go mod download since we don't have external dependencies yet
-  # If we add dependencies later, we can uncomment this
-  # go mod download
+# Function to compare versions
+version_gt() { 
+    test "$(printf '%s\n' "$1" "$2" | sort -V | head -n1)" != "$1"
+}
+
+# Only run go mod commands if we're using Go 1.11 or higher
+if version_gt "$GO_VERSION" "1.10"; then
+    echo -e "${YELLOW}Go modules supported (Go $GO_VERSION).${NC}"
+    
+    # We don't need go mod download since we don't have external dependencies yet
+    # If we add dependencies later, we can uncomment this
+    # echo -e "${YELLOW}Downloading dependencies...${NC}"
+    # go mod download
 else
-  echo -e "${YELLOW}Go modules not supported in this version. Skipping dependency check.${NC}"
+    echo -e "${YELLOW}Go modules not supported in this version (Go $GO_VERSION). Skipping module commands.${NC}"
 fi
 
 # Format check
 echo -e "\n${YELLOW}Checking code formatting...${NC}"
 if [ "$(gofmt -l . | wc -l)" -gt 0 ]; then
-  echo -e "${RED}The following files need formatting:${NC}"
-  gofmt -l .
-  echo -e "${YELLOW}Running gofmt to fix formatting issues...${NC}"
-  gofmt -w .
-  echo -e "${GREEN}Formatting fixed.${NC}"
+    echo -e "${RED}The following files need formatting:${NC}"
+    gofmt -l .
+    echo -e "${YELLOW}Running gofmt to fix formatting issues...${NC}"
+    gofmt -w .
+    echo -e "${GREEN}Formatting fixed.${NC}"
 else
-  echo -e "${GREEN}All files are properly formatted.${NC}"
+    echo -e "${GREEN}All files are properly formatted.${NC}"
 fi
 
 # Lint check
 echo -e "\n${YELLOW}Running linter...${NC}"
 if command -v golint > /dev/null; then
-  golint ./...
+    golint ./...
 else
-  echo -e "${RED}golint not installed. Installing...${NC}"
-  go install golang.org/x/lint/golint@latest
-  $(go env GOPATH)/bin/golint ./...
+    echo -e "${RED}golint not installed. Installing...${NC}"
+    if version_gt "$GO_VERSION" "1.16"; then
+        go install golang.org/x/lint/golint@latest
+    else
+        go get -u golang.org/x/lint/golint
+    fi
+    
+    if command -v golint > /dev/null; then
+        golint ./...
+    else
+        $(go env GOPATH)/bin/golint ./...
+    fi
 fi
 
 # Vet check
@@ -73,25 +89,25 @@ go build -v ./cmd/stremax
 # Run examples
 echo -e "\n${YELLOW}Running example programs...${NC}"
 if [ -f "./stremax" ]; then
-  echo -e "${YELLOW}Running simple.sx${NC}"
-  ./stremax run -file ./examples/simple.sx
-  
-  echo -e "\n${YELLOW}Running arithmetic.sx${NC}"
-  ./stremax run -file ./examples/arithmetic.sx
-  
-  echo -e "\n${YELLOW}Running strings.sx${NC}"
-  ./stremax run -file ./examples/strings.sx
-  
-  echo -e "\n${YELLOW}Running conditionals.sx${NC}"
-  ./stremax run -file ./examples/conditionals.sx
-  
-  echo -e "\n${YELLOW}Running boolean.sx${NC}"
-  ./stremax run -file ./examples/boolean.sx
-  
-  echo -e "\n${YELLOW}Running combined.sx${NC}"
-  ./stremax run -file ./examples/combined.sx
+    echo -e "${YELLOW}Running simple.sx${NC}"
+    ./stremax run -file ./examples/simple.sx
+    
+    echo -e "\n${YELLOW}Running arithmetic.sx${NC}"
+    ./stremax run -file ./examples/arithmetic.sx
+    
+    echo -e "\n${YELLOW}Running strings.sx${NC}"
+    ./stremax run -file ./examples/strings.sx
+    
+    echo -e "\n${YELLOW}Running conditionals.sx${NC}"
+    ./stremax run -file ./examples/conditionals.sx
+    
+    echo -e "\n${YELLOW}Running boolean.sx${NC}"
+    ./stremax run -file ./examples/boolean.sx
+    
+    echo -e "\n${YELLOW}Running combined.sx${NC}"
+    ./stremax run -file ./examples/combined.sx
 else
-  echo -e "${RED}stremax binary not found. Build failed?${NC}"
+    echo -e "${RED}stremax binary not found. Build failed?${NC}"
 fi
 
 echo -e "\n${GREEN}All tests completed!${NC}"