Skip to content
New issue

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

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

Already on GitHub? # to your account

Add log file rotation #1125

Merged
merged 1 commit into from
Nov 18, 2024
Merged

Add log file rotation #1125

merged 1 commit into from
Nov 18, 2024

Conversation

gandarez
Copy link
Member

This PR introduces log file rotation to limit its file size. The default settings are 25MB per file and maximum of 4 backups.

Closes #1063

@gandarez gandarez requested a review from alanhamlett November 18, 2024 20:47
@gandarez gandarez self-assigned this Nov 18, 2024
@gandarez gandarez enabled auto-merge November 18, 2024 20:56
@gandarez gandarez merged commit 97c1b1e into develop Nov 18, 2024
21 checks passed
@gandarez gandarez deleted the feature/log-file-rotation branch November 18, 2024 21:42
Copy link

codecov bot commented Nov 18, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.20%. Comparing base (910dd41) to head (c4bfa8c).
Report is 2 commits behind head on develop.

@@             Coverage Diff             @@
##           develop    #1125      +/-   ##
===========================================
+ Coverage    63.18%   63.20%   +0.01%     
===========================================
  Files          386      386              
  Lines        16881    16882       +1     
===========================================
+ Hits         10666    10670       +4     
+ Misses        5646     5644       -2     
+ Partials       569      568       -1     
Flag Coverage Δ
unittests 63.20% <100.00%> (+0.01%) ⬆️

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

Files with missing lines Coverage Δ
cmd/run.go 47.60% <100.00%> (+1.21%) ⬆️
pkg/log/log.go 50.53% <ø> (ø)

🚨 Try these New Features:

@alanhamlett alanhamlett mentioned this pull request Nov 19, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Limit the size of wakatime.log file
2 participants