Skip to content

fix: Get R CMD build to ignore vignette builds#139

Merged
danielinteractive merged 1 commit intomainfrom
cicdguy-patch-1
Jun 17, 2025
Merged

fix: Get R CMD build to ignore vignette builds#139
danielinteractive merged 1 commit intomainfrom
cicdguy-patch-1

Conversation

@cicdguy
Copy link
Contributor

@cicdguy cicdguy commented Jun 17, 2025

Because rmarkdown tends to install the package from GitHub using no auth, so skip vignette building entirely.

Because `rmarkdown` tends to install the package from GitHub using no auth, so skip vignette building entirely.
@github-actions
Copy link
Contributor

badge

Code Coverage Summary

Filename                 Stmts    Miss  Cover    Missing
---------------------  -------  ------  -------  -----------
R/betadiff.R                59       0  100.00%
R/boundsPostprob.R          44       0  100.00%
R/boundsPredprob.R          56       1  98.21%   83
R/dbetabinom.R              81       3  96.30%   32, 62, 136
R/oc2.R                    162     162  0.00%    93-326
R/oc3.R                    146     146  0.00%    91-308
R/ocPostprob.R             117       0  100.00%
R/ocPostprobDist.R         122       0  100.00%
R/ocPredprob.R             205       0  100.00%
R/ocPredprobDist.R         257       5  98.05%   363-367
R/ocRctPostprobDist.R      166       0  100.00%
R/ocRctPredprobDist.R      302       0  100.00%
R/plotBeta.R                63       5  92.06%   106-110
R/plotBounds.R              52      52  0.00%    33-90
R/plotDecision.R            79      79  0.00%    16-136
R/plotOc.R                  50       2  96.00%   94-95
R/postprob.R                34       1  97.06%   106
R/postprobDist.R            77       1  98.70%   204
R/predprob.R                24       0  100.00%
R/predprobDist.R           140       1  99.29%   269
R/runShinyPhase1b.R          4       4  0.00%    8-13
R/sumbetadiff.R             68      15  77.94%   85-103
R/sumTable.R                36       0  100.00%
TOTAL                     2344     477  79.65%

Diff against main

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

Results for commit: 1d2aa03

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   6m 3s ⏱️
  135 tests   134 ✅ 0 💤 1 ❌
1 301 runs  1 300 ✅ 0 💤 1 ❌

For more details on these failures, see this check.

Results for commit 1d2aa03.

@github-actions
Copy link
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
ocPredprobDist 💔 $89.72$ $+2.94$ $0$ $0$ $0$ $0$
ocRctPredprobDist 💔 $26.93$ $+1.07$ $0$ $0$ $0$ $0$
plotOc 💔 $201.47$ $+7.35$ $+2$ $-2$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
ocPredprobDist 💔 $57.77$ $+1.96$ ocPredprobDist_gives_higher_PrFutility_with_more_futility_looks
plotOc 💔 $53.51$ $+1.68$ h_get_dataframe_oc_gives_correct_results_for_ocPredprobDist_when_relativeDelta_FALSE
plotOc 💔 $46.31$ $+1.77$ h_get_dataframe_oc_gives_correct_results_for_ocPredprobDist_when_relativeDelta_TRUE
plotOc 💔 $99.03$ $+3.61$ plotOc_gives_expected_results_for_ocPredprobDist_with_different_relativeDelta_status

Results for commit 2e7a334

♻️ This comment has been updated with latest results.

@danielinteractive danielinteractive merged commit bfa9a0d into main Jun 17, 2025
22 of 24 checks passed
@danielinteractive danielinteractive deleted the cicdguy-patch-1 branch June 17, 2025 13:52
@audreyyeoCH
Copy link
Collaborator

thank you @cicdguy !

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