Add logging - Githubissues

rlee287 / pyautoupdate

An auto-update API for Python programs. ARCHIVED: The update mechanism here contains no guarantees about code integrity. You may want to look at TUF (The Update Framework) instead.

GNU Lesser General Public License v2.1

14 stars 2 forks source link

Add logging #17

Closed rlee287 closed 7 years ago

rlee287 commented 7 years ago

Closes #11 Will be merged after all logging messages are added in.

codecov-io commented 7 years ago

Current coverage is 88.735% (diff: 95.082%)

Merging #17 into develop will increase coverage by 0.899%

@@            develop        #17   diff @@
==========================================
  Files            18         18          
  Lines           559        577    +18   
  Methods           0          0          
  Messages          0          0          
  Branches         63         64     +1   
==========================================
+ Hits            491        512    +21   
+ Misses           48         45     -3   
  Partials         20         20

Powered by Codecov. Last update 7377404...36367f1

rlee287 commented 7 years ago

All working now, but considerable log noise is still present.

Will try to eliminate before merging.

rlee287 commented 7 years ago

Log noise resolved: each call to log_to_stderr() unconditionally attached another handler. This has been fixed by using get_logger() instead where appropriate.

rlee287 commented 7 years ago

All major logging work done; remaining small issues can be fixed in develop.

PullApprove approval signified.