8000 Return cache directory configuration errors by dajiaohuang · Pull Request #1818 · dominikh/go-tools · GitHub
[go: up one dir, main page]

Skip to content

Return cache directory configuration errors - #1818

Closed
dajiaohuang wants to merge 1 commit into
dominikh:masterfrom
dajiaohuang:fix/cache-default-dir-error
Closed

dajiaohuang wants to merge 1 commit into
dominikh:masterfrom
dajiaohuang:fix/cache-default-dir-error

Conversation

@dajiaohuang
Copy link
Copy Markdown

When STATICCHECK_CACHE is relative, DefaultDir records an error and returns an empty path. initDefaultCache currently ignores that error and calls log.Fatalf after MkdirAll fails, even though Default exposes an error-returning contract and lintcmd propagates it to the CLI. Return from initialization when the cached directory error is set so Default returns the intended error.

@dajiaohuang
Copy link
Copy Markdown
Author

Closing this duplicate after comparing the current patches. The same defect and fix are already covered by #1798. Keeping the existing contribution avoids competing review threads; the contributor branches remain preserved.

@dajiaohuang dajiaohuang closed this Oct 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0