fix: add golangci-lint to pre-commit hook, clarify R2_ENDPOINT in env example
Pre-commit now runs golangci-lint (3m timeout) after go vet, before staticcheck. R2_ENDPOINT comment updated to clarify it is prod-only and local dev uses S3_* vars. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
+2
-1
@@ -23,7 +23,8 @@ S3_BUCKET=crussell
|
|||||||
S3_PROFILE_PICS_BUCKET=crussell-profile-pics
|
S3_PROFILE_PICS_BUCKET=crussell-profile-pics
|
||||||
AWS_REGION=eu-west-2
|
AWS_REGION=eu-west-2
|
||||||
|
|
||||||
# Prod: Set R2_ENDPOINT for Cloudflare R2 (overrides S3_* vars in non-dev builds)
|
# Prod only: Cloudflare R2 (overrides S3_* vars in non-dev builds).
|
||||||
|
# Local dev uses the S3_* vars above (from .env). Not needed for local builds.
|
||||||
R2_ENDPOINT=
|
R2_ENDPOINT=
|
||||||
# R2_ACCESS_KEY=<your-access-key>
|
# R2_ACCESS_KEY=<your-access-key>
|
||||||
# R2_SECRET_KEY=<your-secret-key>
|
# R2_SECRET_KEY=<your-secret-key>
|
||||||
|
|||||||
@@ -41,6 +41,13 @@ if [ -n "$BACKEND_FILES" ]; then
|
|||||||
fi
|
fi
|
||||||
cd ..
|
cd ..
|
||||||
|
|
||||||
|
printf "${YELLOW}Checking golangci-lint...${NC}\n"
|
||||||
|
if ! cd backend && golangci-lint run ./... --timeout 3m 2>&1; then
|
||||||
|
printf "${RED}✖ golangci-lint failed${NC}\n"
|
||||||
|
FAILED=1
|
||||||
|
fi
|
||||||
|
cd ..
|
||||||
|
|
||||||
printf "${YELLOW}Checking staticcheck...${NC}\n"
|
printf "${YELLOW}Checking staticcheck...${NC}\n"
|
||||||
if ! cd backend && staticcheck -tags "test,dev" ./... 2>&1; then
|
if ! cd backend && staticcheck -tags "test,dev" ./... 2>&1; then
|
||||||
printf "${RED}✖ staticcheck failed${NC}\n"
|
printf "${RED}✖ staticcheck failed${NC}\n"
|
||||||
|
|||||||
Reference in New Issue
Block a user