Skip to content

Commit 2ed3d19

Browse files
committed
fix: restore version 2 format and correct golangci-lint configuration
- Add required 'version: 2' field as requested in review - Move goimports and gofmt from linters to formatters section - Fix configuration structure to match golangci-lint v2 format - Maintain all original and new linters from previous enhancement Addresses feedback from maintainer review in #367 Signed-off-by: Rishi Jat <rishijat098@gmail.com>
1 parent 7f7b6e8 commit 2ed3d19

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

.golangci.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,17 @@
1+
version: "2"
2+
13
# Enhanced golangci-lint configuration for llm-d-inference-scheduler
24
# Expanded from 22 to 30+ linters for improved code quality, security, and maintainability
35
# Focus on practical improvements suitable for Kubernetes controller development
46

57
run:
68
timeout: 10m
7-
modules-download-mode: readonly
9+
allow-parallel-runners: true
10+
11+
formatters:
12+
enable:
13+
- goimports
14+
- gofmt
815

916
linters:
1017
enable:
@@ -31,10 +38,6 @@ linters:
3138
- unused # Unused code (helps reduce bloat)
3239

3340
# === NEW HIGH-VALUE ADDITIONS ===
34-
# Formatting and style consistency
35-
- gofmt # Ensures code is gofmt-ed
36-
- goimports # Import organization and unused import removal
37-
3841
# Security enhancements
3942
- gosec # Security vulnerability scanner
4043
- bodyclose # Ensures HTTP response bodies are closed
@@ -70,7 +73,6 @@ linters:
7073
- gocognit # Cognitive complexity (can be overly strict)
7174

7275
issues:
73-
# Exclude certain linters from running on files until build dependencies are resolved
7476
exclude-rules:
7577
- path: pkg/plugins/
7678
linters:

0 commit comments

Comments
 (0)