go-tools
go-tools copied to clipboard
Please include support for zerolog Panic (nil pointer check)
Please add support for zerolog "Panic" to avoid triggering "SA5011: possible nil pointer dereference".
Hi @dominikh Is there any update on this issue?
I think that that was discussed somewhere else, but perhaps a list of fully-qualified identifier names of terminating functions could be added to staticcheck.conf
. So that people could update the for their projects' needs instead of teaching staticcheck
about all logging and utility libraries.
Please file an issue if we’re missing support for a popular package.
I found this in docs.
https://github.com/dominikh/go-tools/blob/82c7dc98d5026ca37b7f82c981597810cf1c37aa/ir/exits.go#L22-L60 And staticcheck only supports logrus. I think zerolog can be added.
@ainar-g Could you give guide how to add terminating functions to configuration?
@dlsrb6342, I cannot provide any guides, since the feature doesn't exist yet. My comment was merely a proposal to add it.