search

Karaoke-Manager / legacy-backend

The Karman API, written in Python/FastAPI
MIT License
1 stars 0 forks source link

Logging #8

Closed codello closed 2 years ago

codello commented 2 years ago

This PR should be merged after #7

This PR adds a logging infrastructure using Python's logging module. It adds a configuration file as well as settings that make it possible to configure logging as a developer or admin.

Fixes #5

codecov[bot] commented 2 years ago

Codecov Report

Merging #8 (b7912c0) into main (57f82b9) will decrease coverage by 7.73%. The diff coverage is 65.54%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #8      +/-   ##
==========================================
- Coverage   86.26%   78.53%   -7.74%     
==========================================
  Files          12       19       +7     
  Lines         284      410     +126     
==========================================
+ Hits          245      322      +77     
- Misses         39       88      +49
Impacted Files Coverage Δ
karman/logging.py 35.38% <35.38%> (ø)
karman/schemas/base.py 94.11% <83.33%> (-5.89%) :arrow_down:
karman/config/helpers.py 84.61% <84.61%> (ø)
karman/config/settings.py 91.83% <84.61%> (ø)
karman/config/__init__.py 100.00% <100.00%> (ø)
karman/models/__init__.py 100.00% <100.00%> (ø)
karman/models/base.py 100.00% <100.00%> (ø)
karman/models/song.py 100.00% <100.00%> (ø)
karman/routes/auth.py 88.88% <100.00%> (+0.65%) :arrow_up:
karman/routes/songs.py 100.00% <100.00%> (ø)
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 57f82b9...b7912c0. Read the comment docs.