AlexStocks commented on issue #3552:
URL: https://github.com/apache/dubbo-go/issues/3552#issuecomment-5099235078

   ## go vet verification (2026-07-28)
   
   Ran `GOTOOLCHAIN=local go vet ./...` with local Go 1.26.2 on current develop 
tip (HEAD 53d81d17).
   
   **Result: EXIT=0, zero warnings.**
   
   ### Coverage note
   The default `go vet` analyzers (printf, structtag, copylocks, lostcancel, 
unreachable, unsafeptr, etc.) did **not** flag any issue in this thread. This 
is expected — `go vet` does not cover:
   - file-descriptor leaks (needs the `bodyclose` linter)
   - goroutine / context-cancellation leaks (needs `noctx` + manual review)
   - unsafe type assertions without comma-ok (only the `assert` analyzer, and 
not these cases)
   - silently ignored errors (needs `errorlint` / `nilerr`)
   
   These are exactly the linters proposed in the enhanced golangci config 
(`docs/golangci-enhanced.example.yml`). Enabling them would surface these 
issues at PR time and prevent regressions.
   
   ### Note on the earlier run
   The first `go vet ./...` used `GOTOOLCHAIN=go1.25.0+auto` and hung 
downloading the 1.25.0 toolchain (network). Switching to the locally-installed 
`go1.26.2` (`GOTOOLCHAIN=local`) completed in seconds via cached build 
artifacts.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to