Skip to content

Save system metrics gathered during R CMD check phase#98

Merged
walkowif merged 17 commits intomainfrom
add-system-debug-information
Feb 21, 2024
Merged

Save system metrics gathered during R CMD check phase#98
walkowif merged 17 commits intomainfrom
add-system-debug-information

Conversation

@walkowif
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Feb 20, 2024

Unit Tests Summary

33 tests   30 ✅  18s ⏱️
 2 suites   3 💤
 1 files     0 ❌

Results for commit 6129264.

♻️ This comment has been updated with latest results.

@github-actions
Copy link

github-actions bot commented Feb 20, 2024

badge

Code Coverage Summary

Filename                   Stmts    Miss  Cover    Missing
-----------------------  -------  ------  -------  ----------------------------------------------------------------------------------------
cmd/check.go                 230     108  53.04%   53-54, 147-149, 166-247, 261-263, 297-326
cmd/dependencies.go          127      17  86.61%   201-217
cmd/download.go              588     183  68.88%   150-274, 280-294, 339-353, 486-488, 511-515, 518-534, 548-550, 558-562, 600-605, 649-657
cmd/install.go               296     201  32.09%   91, 102-143, 167-199, 219-221, 230-233, 240-267, 285-288, 339-452
cmd/renv.go                  125      74  40.80%   97-102, 148-219
cmd/report.go                134      66  50.75%   57-69, 115-117, 135-136, 177-227
cmd/root.go                  277     274  1.08%    69-322, 329-377
cmd/system_metrics.go         78      78  0.00%    33-135
cmd/systeminfo.go             32      19  40.62%   39-43, 63-77
cmd/systeminfo_linux.go       48      33  31.25%   28-30, 46-79
cmd/utils.go                 108      29  73.15%   46-60, 108-115, 137-140, 148-151
main.go                        3       3  0.00%    5-7
TOTAL                       2046    1085  46.97%

Diff against main

Filename                 Stmts    Miss  Cover
---------------------  -------  ------  --------
cmd/check.go                +7      +7  -1.67%
cmd/dependencies.go         +1       0  +0.11%
cmd/root.go                 +7      +7  -0.03%
cmd/system_metrics.go      +78     +78  +100.00%
cmd/utils.go                -1       0  -0.25%
TOTAL                      +92     +92  -2.21%

Results for commit: 6129264

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@walkowif walkowif marked this pull request as ready for review February 20, 2024 18:42
@walkowif walkowif requested a review from a team as a code owner February 20, 2024 18:42
Copy link
Contributor

@cicdguy cicdguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Does this work well with Go 1.22? Not necessary to upgrade, but would be nice to use the newer release.

@walkowif walkowif merged commit 958f54a into main Feb 21, 2024
@walkowif walkowif deleted the add-system-debug-information branch February 21, 2024 11:28
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