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

Workflow propagations #304

Merged
merged 4 commits into from
Sep 16, 2024
Merged

Workflow propagations #304

merged 4 commits into from
Sep 16, 2024

Conversation

walkowif
Copy link
Contributor

@walkowif walkowif requested a review from a team August 29, 2024 07:53
@walkowif walkowif marked this pull request as ready for review September 13, 2024 16:06
Copy link
Contributor

github-actions bot commented Sep 13, 2024

Unit Tests Summary

  1 files  114 suites   34s ⏱️
111 tests 111 ✅ 0 💤 0 ❌
331 runs  331 ✅ 0 💤 0 ❌

Results for commit 6f3f71e.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Sep 13, 2024

Unit Test Performance Difference

Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
add_calculated_row 💚 $1.17$ $-1.07$ add_calculated_row_x_
apply_fmt_fn 💔 $0.04$ $+1.00$ apply_fmt_fn_works
ard_hierarchical 💚 $9.33$ $-1.42$ ard_hierarchical_works_without_by_variables
shuffle_ard 💚 $1.61$ $-1.45$ shuffle_trim_works

Results for commit 2651952

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Sep 13, 2024

badge

Code Coverage Summary

Filename                       Stmts    Miss  Cover    Missing
---------------------------  -------  ------  -------  -------------------------------
R/add_calculated_row.R            46       0  100.00%
R/apply_fmt_fn.R                  98       0  100.00%
R/ard_attributes.R                35       1  97.14%   56
R/ard_categorical.R              348      10  97.13%   233, 390-394, 401-402, 578, 612
R/ard_complex.R                   36       1  97.22%   83
R/ard_continuous.R               176       3  98.30%   200-201, 299
R/ard_dichotomous.R               80       1  98.75%   62
R/ard_hierarchical.R              83       0  100.00%
R/ard_missing.R                   58       1  98.28%   50
R/ard_stack.R                     84       1  98.81%   107
R/ard_total_n.R                   10       0  100.00%
R/as_card.R                        5       0  100.00%
R/as_nested_list.R                41       0  100.00%
R/bind_ard.R                      47      11  76.60%   74-85
R/check_ard_structure.R           39       6  84.62%   31, 55-59
R/default_stat_labels.R           18       0  100.00%
R/eval_capture_conditions.R       18       0  100.00%
R/get_ard_statistics.R            16       0  100.00%
R/nest_for_ard.R                  71       1  98.59%   60
R/print_ard_conditions.R          78       0  100.00%
R/print.R                         72       0  100.00%
R/process_selectors.R            126       1  99.21%   337
R/replace_null_statistic.R        11       0  100.00%
R/round5.R                         1       0  100.00%
R/selectors.R                     19       0  100.00%
R/shift_ard_columns.R             77       1  98.70%   147
R/shuffle_ard.R                  129       0  100.00%
R/summary_functions.R             25       1  96.00%   59
R/tidy_ard_order.R                25       0  100.00%
R/tidy_as_ard.R                   39       0  100.00%
R/utils.R                         24       0  100.00%
TOTAL                           1935      39  97.98%

Diff against main

Filename                  Stmts    Miss  Cover
----------------------  -------  ------  -------
R/apply_fmt_fn.R              0     -24  +24.49%
R/ard_categorical.R           0     -11  +3.16%
R/ard_continuous.R            0      -1  +0.57%
R/ard_dichotomous.R           0      -6  +7.50%
R/ard_stack.R                 0      -6  +7.14%
R/bind_ard.R                  0      -7  +14.89%
R/get_ard_statistics.R        0      -1  +6.25%
R/shift_ard_columns.R         0     -12  +15.58%
R/tidy_as_ard.R               0     -11  +28.21%
TOTAL                         0     -79  +4.08%

Results for commit: 6f3f71e

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@walkowif walkowif enabled auto-merge (squash) September 16, 2024 14:31
@walkowif walkowif merged commit 39c602c into main Sep 16, 2024
30 checks passed
@walkowif walkowif deleted the propagations-september-2024 branch September 16, 2024 14:34
@github-actions github-actions bot locked and limited conversation to collaborators Sep 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants