Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@dni
Copy link
Member

@dni dni commented Apr 22, 2024

just some simply refactoring to clean up app.py

@codecov
Copy link

codecov bot commented Apr 22, 2024

Codecov Report

Attention: Patch coverage is 90.78947% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 60.97%. Comparing base (00f39a2) to head (dfc46fe).

Files Patch % Lines
lnbits/utils/logger.py 89.85% 7 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2454      +/-   ##
==========================================
+ Coverage   60.94%   60.97%   +0.02%     
==========================================
  Files          65       66       +1     
  Lines        9395     9404       +9     
==========================================
+ Hits         5726     5734       +8     
- Misses       3669     3670       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dni dni force-pushed the refactor-logger branch from aa310e2 to dfc46fe Compare April 22, 2024 11:13
@dni dni requested review from arcbtc, callebtc, motorina0 and prusnak and removed request for callebtc and prusnak April 24, 2024 05:48
Copy link
Member

@arcbtc arcbtc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lovely

@arcbtc arcbtc merged commit b9e62bf into dev May 9, 2024
@arcbtc arcbtc deleted the refactor-logger branch May 9, 2024 16:51
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.

3 participants