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

add sparkline::sparkline to helper-TealAppDriver.R #867

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

m7pr
Copy link
Contributor

@m7pr m7pr commented Feb 27, 2025

@m7pr m7pr requested a review from donyunardi February 27, 2025 15:27
@m7pr m7pr added the core label Feb 27, 2025
Copy link
Contributor

github-actions bot commented Feb 27, 2025

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ---------------------------------------
R/tm_a_pca.R                    889     889  0.00%    139-1161
R/tm_a_regression.R             777     777  0.00%    178-1060
R/tm_data_table.R               211     211  0.00%    100-359
R/tm_file_viewer.R              173     173  0.00%    47-255
R/tm_front_page.R               144     133  7.64%    77-247
R/tm_g_association.R            346     346  0.00%    159-580
R/tm_g_bivariate.R              690     426  38.26%   331-818, 859, 970, 987, 1005, 1016-1038
R/tm_g_distribution.R          1119    1119  0.00%    156-1422
R/tm_g_response.R               369     369  0.00%    177-625
R/tm_g_scatterplot.R            731     731  0.00%    260-1095
R/tm_g_scatterplotmatrix.R      296     277  6.42%    198-531, 592, 606
R/tm_missing_data.R            1133    1133  0.00%    124-1434
R/tm_outliers.R                1041    1041  0.00%    163-1355
R/tm_t_crosstable.R             261     261  0.00%    163-472
R/tm_variable_browser.R         832     827  0.60%    89-1078, 1116-1299
R/utils.R                       151     135  10.60%   87-272, 302-338, 350-359, 364, 378-397
R/zzz.R                           2       2  0.00%    2-3
TOTAL                          9165    8850  3.44%

Diff against main

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

Results for commit: dadb577

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Feb 27, 2025

Unit Tests Summary

  1 files   22 suites   13m 14s ⏱️
144 tests 107 ✅ 37 💤 0 ❌
474 runs  437 ✅ 37 💤 0 ❌

Results for commit dadb577.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Feb 27, 2025

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
shinytest2-tm_a_pca 💚 $120.61$ $-1.79$ $0$ $0$ $0$ $0$
shinytest2-tm_g_bivariate 💔 $74.05$ $+1.11$ $0$ $0$ $0$ $0$
shinytest2-tm_outliers 💔 $111.07$ $+2.44$ $0$ $0$ $0$ $0$
shinytest2-tm_t_crosstable 💔 $31.43$ $+1.21$ $0$ $0$ $0$ $0$
shinytest2-tm_variable_browser 💔 $56.16$ $+2.32$ $0$ $0$ $0$ $0$

Results for commit e6c3944

♻️ This comment has been updated with latest results.

Signed-off-by: Marcin <[email protected]>
@donyunardi
Copy link
Contributor

Let's not merge until I pushed tmg to CRAN.
The failures are for shinytest2, which are not blockers for CRAN release.

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.

2 participants