search

go-simpler / sloglint

🪵 Ensure consistent code style when using log/slog
https://go-simpler.org/sloglint
Mozilla Public License 2.0
75 stars 5 forks source link

feat: support `With` and `LogAttrs` functions #41

Closed tigrato closed 2 months ago

tigrato commented 2 months ago

This PR enforces the lint rules to With and LogAttrs functions that are enforced to arguments in other logger funcs.

Fixes https://github.com/go-simpler/sloglint/issues/38

codecov[bot] commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.80%. Comparing base (f87bbdf) to head (9e40331). Report is 5 commits behind head on main.

:exclamation: Current head 9e40331 differs from pull request most recent head 9bbec78. Consider uploading reports for the commit 9bbec78 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #41 +/- ## ========================================== - Coverage 84.06% 83.80% -0.26% ========================================== Files 2 2 Lines 251 315 +64 ========================================== + Hits 211 264 +53 - Misses 33 42 +9 - Partials 7 9 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

tmzane commented 2 months ago

Thanks!

tigrato commented 2 months ago

@tmzane can you please cut a new release so we can integrate it with our CI?

tmzane commented 2 months ago

Yep, just wanted to refactor some internals first. Almost done.

tmzane commented 2 months ago

@tigrato see https://github.com/go-simpler/sloglint/releases/tag/v0.7.0