Skip to content

Add log rotation using lumberjack#115

Open
njharter wants to merge 3 commits intomainfrom
log-rotation-00
Open

Add log rotation using lumberjack#115
njharter wants to merge 3 commits intomainfrom
log-rotation-00

Conversation

@njharter
Copy link
Copy Markdown

What's Changing

  • Add log rotation using lumberjack.

@njharter njharter requested review from piccione99 and renaz6 March 31, 2026 21:58
@njharter njharter self-assigned this Mar 31, 2026
@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


nharte968_comcast seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.65%. Comparing base (a438a65) to head (9184cfa).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #115      +/-   ##
==========================================
+ Coverage   80.57%   80.65%   +0.07%     
==========================================
  Files          30       30              
  Lines        1797     1804       +7     
==========================================
+ Hits         1448     1455       +7     
  Misses        318      318              
  Partials       31       31              
Flag Coverage Δ
unittests 80.65% <100.00%> (+0.07%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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