Skip to content

Change decimal to 64 bits#91

Merged
syyyr merged 1 commit into
masterfrom
shorten-decimal
Dec 25, 2025
Merged

Change decimal to 64 bits#91
syyyr merged 1 commit into
masterfrom
shorten-decimal

Conversation

@syyyr

@syyyr syyyr commented Dec 25, 2025

Copy link
Copy Markdown
Contributor

We do not need such precision, the test was a GPS value.

We do not need such precision, the test was a GPS value.
@syyyr syyyr requested a review from fvacek December 25, 2025 21:47
@github-actions

Copy link
Copy Markdown

Code coverage report for 2d8305e

Coverage difference

❌ Code coverage worsened.

Filename Stmts Miss Cover Missing
src/decimal.rs +9 +2 +2.18% 70, 71, 72, 75, 76, 80, 81
TOTAL +9 +2 -0.01%

Overall coverage

Click to expand
Filename Stmts Miss Cover Missing
src/chainpack.rs 635 14 97.80% 87, 120, 270-282, 287
src/rpcvalue.rs 686 178 74.05% 70, 72-76, 78, 80-81, 84-99, 126-128, 156-158, 174-176, 216-218, 257-262, 299-301, 413-425, 433, 450, 457-462, 480, 487-512, 520, 532, 559, 576, 593, 603, 613, 623, 643, 653, 663, 673, 683, 697-726, 926-943, 982-984, 988, 998, 1004-1013, 1017, 1020-1026, 1030, 1033-1038, 1042, 1054, 1066, 1072, 1078, 1081-1096, 1118-1123, 1135-1152
src/json.rs 411 23 94.40% 221, 257, 265, 278, 319-333, 349, 394-396, 408-409, 518
src/lib.rs 9 0 100.00%
src/datetime.rs 161 59 63.35% 22-27, 91-95, 103, 109-114, 118-120, 141, 146-148, 154-155, 165-166, 181, 185-232
src/metamap.rs 106 34 67.92% 44-51, 67-72, 86-93, 99, 126-143
src/textrdwr.rs 234 30 87.18% 47-72, 97, 132, 135, 145, 148, 160, 168-170, 213, 229, 241, 251, 267, 309
src/writer.rs 18 0 100.00%
src/reader.rs 64 5 92.19% 20-22, 65, 83
src/cpon.rs 526 85 83.84% 35-37, 41, 45-47, 55, 60, 65-67, 80, 84-86, 100, 107-114, 133-154, 166-205, 308-309, 337-338, 344, 366, 379-383, 413, 429, 455-460, 497, 499-503, 518, 640
src/util.rs 84 43 48.81% 24, 31-77
src/decimal.rs 72 27 62.50% 30-31, 51-54, 59-63, 65, 69-86
libshvproto-macros/src/lib.rs 291 30 89.69% 13, 24, 27, 41, 111, 115-118, 135, 239, 259, 262-268, 277-281, 295, 303-306, 399, 435, 494
src/bin/cp2cp.rs 100 36 64.00% 97-98, 118-124, 137, 141-142, 146-176
TOTAL 3397 564 83.40%

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/libshvproto-rs download -n libshvproto-cov-html-2d8305e6d7adca38f1889bafdb6f0df2774bd779 && ls -1 | xargs -n1 xdg-open)

You can download the HTML reports manually here.

@syyyr syyyr merged commit 9dca727 into master Dec 25, 2025
10 checks passed
@syyyr syyyr deleted the shorten-decimal branch December 25, 2025 21:50
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