Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: improve logging #17

Merged
merged 5 commits into from
Feb 21, 2024
Merged

feat: improve logging #17

merged 5 commits into from
Feb 21, 2024

Conversation

heinrich10
Copy link
Owner

@heinrich10 heinrich10 commented Feb 21, 2024

What?

improve logging across the app

Why?

  • access log can be improved
  • debug logs not shown

How?

configure logging instance

Anything Else?

move config to utils

Copy link

github-actions bot commented Feb 21, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
291 277 95% 0% 🟢

New Files

File Coverage Status
pyfastapi/utils/logging.py 73% 🟢
TOTAL 73% 🟢

Modified Files

File Coverage Status
pyfastapi/init.py 100% 🟢
pyfastapi/controllers/person.py 92% 🟢
pyfastapi/libs/db.py 100% 🟢
pyfastapi/main.py 100% 🟢
pyfastapi/utils/init.py 100% 🟢
TOTAL 98% 🟢

updated for commit: e548076 by action🐍

@heinrich10 heinrich10 merged commit 4eb793f into main Feb 21, 2024
1 check passed
@heinrich10 heinrich10 deleted the feat/improve-logging branch February 21, 2024 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant