Skip to content

Fix clippy warnings in json tests #53

New issue

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

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

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 20, 2025
Merged

Conversation

j4r0u53k
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Mar 18, 2025

Code coverage report for 61a40c8

Coverage difference

All new changes were covered.

Filename Stmts Miss Cover Missing
src/json.rs -1 0 -0.01%
TOTAL -1 0 -0.00%

Overall coverage

Click to expand
Filename Stmts Miss Cover Missing
libshvproto-macros/src/lib.rs 390 31 92.05% 13, 24, 27, 41, 110, 114-117, 131, 241, 265, 268-275, 283-288, 301, 312-316, 446
src/writer.rs 18 0 100.00%
src/datetime.rs 162 59 63.58% 22-27, 80-84, 92, 98-103, 107-109, 130, 135-137, 143-144, 154-155, 170, 174-221
src/rpcvalue.rs 695 180 74.10% 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/util.rs 83 43 48.19% 23, 30-76
src/decimal.rs 66 27 59.09% 26-27, 47-50, 55-59, 61, 65-80
src/textrdwr.rs 239 30 87.45% 47-72, 97, 132, 135, 145, 148, 160, 168-170, 215, 231, 243, 253, 269, 311
src/metamap.rs 111 35 68.47% 44-51, 67-72, 86-93, 99, 126-143
src/lib.rs 9 0 100.00%
src/chainpack.rs 649 15 97.69% 87, 120, 270-282, 287, 524
src/json.rs 420 26 93.81% 131, 147, 225, 253, 263, 271, 284, 325-339, 355, 400-402, 414-415, 524
src/cpon.rs 520 81 84.42% 36, 40-42, 52, 57-59, 72, 76-78, 92, 99-106, 125-146, 158-197, 300-301, 329-330, 336, 358, 371-375, 405, 421, 447-452, 489, 491-495, 510, 632
src/reader.rs 64 5 92.19% 20-22, 65, 83
src/bin/cp2cp.rs 106 39 63.21% 19-24, 95-96, 116-122, 135, 139-140, 144-173
TOTAL 3532 571 83.83%

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-61a40c870f8ac8cc32c831c314f0a3f1aa0374b7 && ls -1 | xargs -n1 xdg-open)

You can download the HTML reports manually here.

@j4r0u53k j4r0u53k requested a review from syyyr March 18, 2025 12:38
@j4r0u53k j4r0u53k force-pushed the fix-clippy-warnings-json branch from f0c36db to 61a40c8 Compare March 19, 2025 16:04
@j4r0u53k j4r0u53k requested a review from syyyr March 19, 2025 16:05
@fvacek fvacek merged commit 8c2be7c into master Mar 20, 2025
6 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.

3 participants