Initial logging - Githubissues

sonatype-nexus-community / nancy

A tool to check for vulnerabilities in your Golang dependencies, powered by Sonatype OSS Index

Apache License 2.0

564 stars 74 forks source link

Initial logging #94

Closed DarthHater closed 4 years ago

DarthHater commented 4 years ago

Welp, it's about time we got better about logging!

This pull request makes the following changes:

Adds a logger package, so you can obtain a logger from anywhere
Partially implements the logger for the IQ path (thus far)

It relates to the following issue #s:

Fixes #76

cc @bhamail / @DarthHater / @zendern / @fitzoh

zendern commented 4 years ago

Overall I like it and it looks good....just dropped some questions and thoughts throughout.

DarthHater commented 4 years ago

Marking this as ready for review. It doesn't have logging EVERYWHERE yet, but I also don't know if I want to balloon this PR anymore. We can create Issues for implementing logging in additional packages, etc...

@zendern @bhamail @fitzoh LET THE TEARING OF MY WORK APART BEGIN!