Require cache to be instantiated with a logger - Githubissues

OpsLevel / opslevel-go

A Go client for interacting with OpsLevel

MIT License

5 stars 13 forks source link

Require cache to be instantiated with a logger #383

Closed taimoorgit closed 6 months ago

taimoorgit commented 7 months ago

Issues

https://github.com/OpsLevel/team-platform/issues/214

Improve code quality by removing the global variable and creating an instantiation function for cache.

Also allow library consumer to pass in a logger rather than being forced to use ours.

Changelog

[x] Add NewCacher(), allows passing in a zerolog.Logger
[x] Log errors in a structured field using .Err(err)
[x] Make a changie entry

Tophatting