sloglint icon indicating copy to clipboard operation
sloglint copied to clipboard

feat: support `slog.Group` function

Open romanchechyotkin opened this issue 11 months ago • 3 comments

Added support for slog.Group

Since it has pairs to check, i decided to add func log/slog.Group to list of slog functions to make it checkable)

romanchechyotkin avatar Jan 06 '25 21:01 romanchechyotkin

@tmzane i will be pleased to get any feedback

romanchechyotkin avatar Jan 06 '25 22:01 romanchechyotkin

Thanks for the PR!

I'm not sure it's safe to just add slog.Group to the list: it doesn't have the alternative function with the context (so it should not be checked for context-only), it doesn't have the message argument (so it should not be checked for static-msg), probably something else. I'll do some research and update your PR.

tmzane avatar Feb 04 '25 20:02 tmzane

let me know any updates

romanchechyotkin avatar Feb 04 '25 20:02 romanchechyotkin