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

Adds roxy.shinylive to pre-commit configuration #793

Merged
merged 3 commits into from
Nov 4, 2024
Merged

Conversation

averissimo
Copy link
Contributor

Pull Request

Fixes insightsengineering/nestdevs-tasks#85

Changes description

  • Add dependency to pre-commit configuration

Signed-off-by: André Veríssimo <211358+averissimo@users.noreply.github.com>
@averissimo averissimo added the core label Nov 1, 2024
@averissimo averissimo changed the title Update .pre-commit-config.yaml Adds roxy.shinylive to pre-commit configuration Nov 1, 2024
Copy link
Contributor

github-actions bot commented Nov 1, 2024

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  ------------------------------------
R/tm_a_pca.R                    827     827  0.00%    109-1069
R/tm_a_regression.R             773     773  0.00%    152-1030
R/tm_data_table.R               185     185  0.00%    102-341
R/tm_file_viewer.R              173     173  0.00%    48-256
R/tm_front_page.R               133     122  8.27%    74-232
R/tm_g_association.R            330     330  0.00%    136-538
R/tm_g_bivariate.R              672     410  38.99%   303-769, 810, 921, 938, 956, 967-989
R/tm_g_distribution.R          1048    1048  0.00%    127-1313
R/tm_g_response.R               351     351  0.00%    153-577
R/tm_g_scatterplot.R            722     722  0.00%    235-1058
R/tm_g_scatterplotmatrix.R      278     259  6.83%    174-481, 542, 556
R/tm_missing_data.R            1069    1069  0.00%    93-1318
R/tm_outliers.R                 985     985  0.00%    131-1260
R/tm_t_crosstable.R             251     251  0.00%    140-439
R/tm_variable_browser.R         830     825  0.60%    91-1083, 1121-1305
R/utils.R                        99      96  3.03%    82-267
R/zzz.R                           2       2  0.00%    2-3
TOTAL                          8728    8428  3.44%

Diff against main

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

Results for commit: 94ece63

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Nov 1, 2024

Unit Tests Summary

  1 files   22 suites   12m 52s ⏱️
147 tests 147 ✅ 0 💤 0 ❌
479 runs  479 ✅ 0 💤 0 ❌

Results for commit 94ece63.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Nov 1, 2024

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
shinytest2-tm_variable_browser 💚 $60.52$ $-1.10$ $0$ $0$ $0$ $0$

Results for commit 8020c07

♻️ This comment has been updated with latest results.

@pawelru pawelru self-assigned this Nov 1, 2024
@averissimo averissimo enabled auto-merge (squash) November 4, 2024 12:09
@averissimo averissimo disabled auto-merge November 4, 2024 12:25
@averissimo averissimo enabled auto-merge (squash) November 4, 2024 12:29
@averissimo averissimo merged commit b0d1863 into main Nov 4, 2024
27 checks passed
@averissimo averissimo deleted the 85_precommit@main branch November 4, 2024 12:53
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: roxy.shinylive is not included in .pre-commit-config.yaml
2 participants