Skip to content
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

Update rtables staged dep #183

Merged
merged 3 commits into from
Feb 22, 2023
Merged

Update rtables staged dep #183

merged 3 commits into from
Feb 22, 2023

Conversation

edelarua
Copy link
Contributor

Closes #182

@edelarua edelarua added the sme label Feb 22, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 22, 2023

badge

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  ----------------------------------------------------------------------------------
R/AddCardModule.R       140       1  99.29%   195
R/Archiver.R             25       0  100.00%
R/ContentBlock.R         16       0  100.00%
R/DownloadModule.R      207      49  76.33%   89-95, 138, 163-168, 177-181, 184-188, 196-200, 203-207, 214-218, 221-225, 262-266
R/FileBlock.R            13       0  100.00%
R/NewpageBlock.R          2       0  100.00%
R/PictureBlock.R         30       2  93.33%   15, 101
R/Previewer.R           292      54  81.51%   183, 199, 201-207, 210-216, 325-369
R/RcodeBlock.R           15       0  100.00%
R/Renderer.R             81      13  83.95%   158, 160-172, 198, 202
R/ReportCard.R           78       4  94.87%   72-73, 223, 244
R/Reporter.R             96       1  98.96%   255
R/ResetModule.R          55       0  100.00%
R/SimpleReporter.R       29       0  100.00%
R/TableBlock.R            8       0  100.00%
R/TealReportCard.R       31       0  100.00%
R/TextBlock.R            13       0  100.00%
R/utils.R                78      66  15.38%   7, 38-97, 99, 102-109
R/yaml_utils.R           74       2  97.30%   65, 263
TOTAL                  1283     192  85.04%

Diff against main

Filename      Stmts    Miss  Cover
----------  -------  ------  --------
TOTAL             0       0  +100.00%

Results for commit: c9175c9

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions
Copy link
Contributor

Unit Tests Summary

    1 files    19 suites   8s ⏱️
210 tests 210 ✔️ 0 💤 0
359 runs  359 ✔️ 0 💤 0

Results for commit 1fc868e.

@github-actions
Copy link
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
Archiver 💚 $3.12$ $-1.09$ $0$ $0$ $0$ $0$

Results for commit c9175c9

♻️ This comment has been updated with latest results.

@ayogasekaram ayogasekaram self-requested a review February 22, 2023 19:31
Copy link

@ayogasekaram ayogasekaram left a comment

Choose a reason for hiding this comment

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

lgtm @edelarua!

@edelarua edelarua merged commit dd2ddb1 into main Feb 22, 2023
@edelarua edelarua deleted the sd_rtables@main branch February 22, 2023 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

update staged.deps, roche/rtables to insightsengineering/rtables
2 participants