Skip to content

Commit de71a8f

Browse files
blink-so[bot]f0ssel
andcommitted
remove golangci-lint fallback from lint target
Simplified the lint target to just run golangci-lint directly instead of falling back to go vet when golangci-lint is not available. Before: - Complex shell logic to check if golangci-lint exists - Fallback to go vet if not found - Confusing behavior (different tools run depending on environment) After: - Simple: golangci-lint run - Clear error if golangci-lint not installed - Consistent behavior across environments Updated RELEASES.md to document the golangci-lint requirement. Tested: make lint fails cleanly with clear error when golangci-lint missing. Co-authored-by: f0ssel <[email protected]>
1 parent a18b9b3 commit de71a8f

File tree

2 files changed

+3
-6
lines changed

2 files changed

+3
-6
lines changed

Makefile

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -71,10 +71,5 @@ fmt:
7171
.PHONY: lint
7272
lint:
7373
@echo "Linting code..."
74-
@if command -v golangci-lint >/dev/null 2>&1; then \
75-
golangci-lint run; \
76-
else \
77-
echo "golangci-lint not found, running go vet instead..."; \
78-
go vet ./...; \
79-
fi
74+
golangci-lint run
8075
@echo "✓ Linting complete!"

RELEASES.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,8 @@ make fmt
6969
make lint
7070
```
7171

72+
**Note**: The `lint` target requires [golangci-lint](https://golangci-lint.run/) to be installed.
73+
7274
### Quick Build
7375

7476
Build for your current platform:

0 commit comments

Comments
 (0)