Skip to content

fix(config): ensure apps.json is writeable - #4249

Merged
ReenigneArcher merged 1 commit into
LizardByte:masterfrom
SuperSandro2000:config-writeable
Oct 26, 2025
Merged

ReenigneArcher merged 1 commit into
LizardByte:masterfrom
SuperSandro2000:config-writeable

Make sure the config is writable after being copied

3333b36
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 26, 2025 in 0s

12.12% (+0.02%) compared to a7f03c2

View this Pull Request on Codecov

12.12% (+0.02%) compared to a7f03c2

Details

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 12.12%. Comparing base (a7f03c2) to head (3333b36).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/config.cpp 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4249      +/-   ##
==========================================
+ Coverage   12.10%   12.12%   +0.02%     
==========================================
  Files          87       87              
  Lines       17610    17612       +2     
  Branches     8096     8097       +1     
==========================================
+ Hits         2131     2136       +5     
+ Misses      14756    14753       -3     
  Partials      723      723              
Files with missing lines Coverage Δ
src/config.cpp 0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes