humanlogio / humanlog

Logs for humans to read.
Apache License 2.0
748 stars 53 forks source link

Nested fields flattened into an object and test cases #126

Closed ferh6t closed 1 week ago

ferh6t commented 1 week ago

Related Issue(s) Related to issue #114

Added small function for nested objects for json logs. Test cases also includes an example for nested object.

aybabtme commented 1 week ago

thanks @ferh6t ☺️ do you mind fixing the failing tests?

aybabtme commented 1 week ago

@ferh6t I think this will work, but you'll need to rebase against the master branch to pick up the change to golangci-lint, similar to https://github.com/humanlogio/humanlog/pull/127#issuecomment-2480900552.

aybabtme commented 1 week ago

actually rerunning it made it pass. when merged to master, i'll run with the --fast flag so this is good to merge. thanks @ferh6t !