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

NEWS #140

Merged
merged 2 commits into from
Oct 6, 2022
Merged

NEWS #140

merged 2 commits into from
Oct 6, 2022

Conversation

mhallal1
Copy link
Contributor

@mhallal1 mhallal1 commented Oct 5, 2022

closes #136

@mhallal1 mhallal1 added the core label Oct 5, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2022

Unit Tests Summary

    1 files    18 suites   11s ⏱️
193 tests 193 ✔️ 0 💤 0
280 runs  280 ✔️ 0 💤 0

Results for commit 4962b3e.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 5, 2022

badge

Code Coverage Summary

Filename              Stmts    Miss  Cover    Missing
------------------  -------  ------  -------  --------------------------------------------------------------------
R/AddCardModule.R       101       0  100.00%
R/Archiver.R             25       0  100.00%
R/ContentBlock.R         16       0  100.00%
R/DownloadModule.R      169      43  74.56%   83-89, 158-163, 172-176, 179-183, 191-195, 198-202, 209-213, 216-220
R/FileBlock.R            13       0  100.00%
R/NewpageBlock.R          2       0  100.00%
R/PictureBlock.R         30       1  96.67%   15
R/Previewer.R           262      52  80.15%   162, 164-167, 172-178, 303-347
R/Renderer.R             54       4  92.59%   142, 144-145, 166
R/ReportCard.R           76       2  97.37%   211, 232
R/Reporter.R             96       1  98.96%   255
R/ResetModule.R          55       0  100.00%
R/SimpleReporter.R       24       0  100.00%
R/TableBlock.R            8       0  100.00%
R/TealReportCard.R       28       0  100.00%
R/TextBlock.R            13       0  100.00%
R/utils.R                 4       1  75.00%   7
R/yaml_utils.R           74       2  97.30%   65, 263
TOTAL                  1050     106  89.90%

Results for commit: be67d42

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@donyunardi donyunardi self-assigned this Oct 6, 2022
Copy link
Contributor

@donyunardi donyunardi left a comment

Choose a reason for hiding this comment

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

lgtm

@mhallal1 mhallal1 merged commit 2b23769 into main Oct 6, 2022
@mhallal1 mhallal1 deleted the 136_NEWS@main branch October 6, 2022 11:04
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.

[UAT] NEWS.md
2 participants