Skip to content

Unify configs#65

Merged
fvacek merged 2 commits into
mainfrom
unify-configs
Jan 13, 2026
Merged

Unify configs#65
fvacek merged 2 commits into
mainfrom
unify-configs

Conversation

@j4r0u53k
Copy link
Copy Markdown
Collaborator

Remove --hp-config option and use just --config. HP specific options can be overriden from command line.

Remove `--hp-config` option and use just `--config`.
HP specific options can be overriden from command line.
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jan 13, 2026

Code coverage report for e9ffb75

Coverage difference

❌ Code coverage worsened.

Filename Stmts Miss Cover Missing
src/bin/hp.rs +4 +4 +100.00% 83, 84, 86, 87, 90, 92, 93, 94, 95, 96, 99, 100, 104, 105, 106, 107, 108, 110, 125
src/lib.rs +12 +10 +3.64% 32, 34, 53, 54, 55, 56, 57, 58, 59, 60, 61
src/sync.rs -3 0 -0.18%
TOTAL +13 +14 -0.19%

Overall coverage

Click to expand
Filename Stmts Miss Cover Missing
src/bin/hp.rs 55 55 0.00% 59-128
src/dirtylog.rs 423 22 94.80% 73, 96-100, 107-108, 123, 132-134, 148-152, 189-193, 201-202, 233
src/getlog.rs 675 91 86.52% 34-133, 245, 255-261, 306, 310-314
src/alarmlog.rs 64 64 0.00% 29-103
src/datachange.rs 110 6 94.55% 79-83, 88
src/cleanup.rs 54 26 51.85% 53-96
src/lib.rs 55 53 3.64% 32-133
src/pushlog.rs 107 107 0.00% 21-164
src/sync.rs 651 255 60.83% 61-62, 72, 80, 88, 97, 148, 150-151, 170, 175, 191, 234-240, 315-332, 339-341, 352-361, 436, 484-706, 727, 786, 793-816, 826, 831, 868-887, 891, 895-905
src/tree.rs 412 412 0.00% 73-689
src/typeinfo.rs 827 265 67.96% 71-73, 102-104, 112, 119-120, 133, 137, 145, 158-211, 217-219, 225-243, 263, 276, 291-293, 297-299, 324-339, 354-355, 372-373, 384-389, 395, 403, 437-438, 459-465, 510-523, 588-607, 632-634, 661-672, 698-699, 710-740, 770-772, 793-795, 809-821, 929-931, 938, 948, 958-960, 968-974, 994-1009, 1030-1038, 1050-1059, 1071, 1094-1096, 1113, 1150, 1153, 1158-1159, 1196-1197, 1200-1201, 1208, 1228-1230, 1234-1238, 1242-1244
src/util.rs 291 53 81.79% 35, 62-63, 66-69, 71, 96, 137-183, 197, 214, 236, 241-249, 253, 264, 271, 278, 305, 319, 456-461
src/sites.rs 593 123 79.26% 60, 168-200, 212-213, 233-265, 296-305, 378, 414-415, 423-424, 429-436, 494-500, 531-549, 558, 588-590, 594, 598, 611-614, 620-622, 626, 641-652
src/journalrw.rs 540 107 80.19% 78, 102, 143, 165, 172, 181, 195, 203-206, 213, 236-244, 260, 288-290, 304-341, 363-368, 372, 375, 389, 392-396, 401, 405-406, 411-412, 416-417, 421-422, 443-445, 465-466, 488-514, 524-525, 529-530, 534-535, 539-540, 544-545, 549-550, 554-555, 559-560, 570-571, 575, 580-581, 696
src/alarm.rs 236 86 63.56% 28-38, 44-46, 49-50, 56-124, 140-142, 149-151, 156, 162, 166-174, 183, 224, 254
TOTAL 5093 1725 66.13%

HTML reports

You can use GitHub CLI to download HTML coverage reports and open them automatically using this command:

(cd "$(mktemp -d)" && gh run -R silicon-heaven/historyprovider-rs download -n historyprovider-rs-cov-html-e9ffb75f86d03ab2f52b94e78ce76d38090fc182 && ls -1 | xargs -n1 xdg-open)

You can download the HTML reports manually here.

@j4r0u53k j4r0u53k requested review from fvacek and syyyr January 13, 2026 16:11
Comment thread src/lib.rs Outdated
Centralize default values for sync limits, journal size, and intervals.
@j4r0u53k j4r0u53k requested a review from fvacek January 13, 2026 16:43
@fvacek fvacek enabled auto-merge January 13, 2026 16:44
@fvacek fvacek merged commit f88d4cf into main Jan 13, 2026
4 checks passed
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.

2 participants