fix: update golangci-lint configuration and CI version

- Remove version field temporarily to test with current CI setup
- Fix incorrect version v2.1.6 in CI workflow
- Use golangci-lint-action@v8 default version

Signed-off-by: Rishi Jat <rishijat098@gmail.com>

Author: rishi-jat

.github/workflows/ci-pr-checks.yaml

@@ -39,7 +39,6 @@ jobs:
       - name: Run lint checks
         uses: golangci/golangci-lint-action@v8
         with:
-          version: 'v2.1.6'
           args: "--config=./.golangci.yml"
 
       - name: Run make test

.golangci.yml

@@ -1,18 +1,11 @@
-version: "2"
-
 # Enhanced golangci-lint configuration for llm-d-inference-scheduler
-# Expanded from 22 to 30+ linters for improved code quality, security, and maintainability
+# Expanded from 22 to 30+ linters for improved code quality, security, and maintainability  
 # Focus on practical improvements suitable for Kubernetes controller development
 
 run:
   timeout: 10m
   allow-parallel-runners: true
 
-formatters:
-  enable:
-    - goimports
-    - gofmt
-
 linters:
   enable:
     # === ORIGINAL LINTERS (maintained) ===
@@ -38,6 +31,10 @@ linters:
     - unused            # Unused code (helps reduce bloat)
 
     # === NEW HIGH-VALUE ADDITIONS ===
+    # Formatting and style consistency
+    - gofmt             # Ensures code is gofmt-ed
+    - goimports         # Import organization and unused import removal
+    
     # Security enhancements
     - gosec             # Security vulnerability scanner
     - bodyclose         # Ensures HTTP response bodies are closed
@@ -71,12 +68,3 @@ linters:
     - funlen            # Function length (can be overly strict)
     - nestif            # Nested if statements (can be overly strict)
     - gocognit          # Cognitive complexity (can be overly strict)
-
-issues:
-  exclude-rules:
-    - path: pkg/plugins/
-      linters:
-        - goanalysis_metalinter
-    - path: pkg/scheduling/pd/
-      linters:
-        - goanalysis_metalinter