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 #284

Merged
merged 10 commits into from
Aug 22, 2024
Merged

Workflow propagations #284

merged 10 commits into from
Aug 22, 2024

Conversation

@walkowif walkowif requested a review from a team August 1, 2024 10:42
Copy link
Contributor

github-actions bot commented Aug 1, 2024


🎉 Thank you for your contribution! Before this PR can be accepted, we require that you all read and agree to our Contributor License Agreement.
You can digitally sign the CLA by posting a comment on this Pull Request in the format shown below. This agreement will apply to this PR as well as all future contributions on this repository.


I have read the CLA Document and I hereby sign the CLA


1 out of 2 committers have signed the CLA.
✅ (walkowif)[https://github.com/walkowif]
@cicdguy
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@walkowif
Copy link
Contributor Author

walkowif commented Aug 1, 2024

I have read the CLA Document and I hereby sign the CLA

@walkowif walkowif marked this pull request as ready for review August 1, 2024 14:15
Copy link
Contributor

github-actions bot commented Aug 1, 2024

Unit Tests Summary

  1 files  106 suites   27s ⏱️
103 tests  67 ✅ 36 💤 0 ❌
227 runs  175 ✅ 52 💤 0 ❌

Results for commit 69a24ff.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 1, 2024

badge

Code Coverage Summary

Filename                       Stmts    Miss  Cover    Missing
---------------------------  -------  ------  -------  -----------------------------------
R/apply_fmt_fn.R                  95      24  74.74%   36-43, 109-116, 198-205
R/ard_attributes.R                35       1  97.14%   56
R/ard_categorical.R              348      21  93.97%   233, 390-394, 401-402, 563-578, 612
R/ard_complex.R                   36       1  97.22%   83
R/ard_continuous.R               176       4  97.73%   118, 200-201, 299
R/ard_dichotomous.R               80       8  90.00%   62, 150-155, 178
R/ard_hierarchical.R              83       0  100.00%
R/ard_missing.R                   58       1  98.28%   50
R/ard_stack.R                     84       7  91.67%   107, 186-191
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                      22       3  86.36%   51-53
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       1  93.75%   75
R/nest_for_ard.R                  71       1  98.59%   60
R/print_ard_conditions.R          75       1  98.67%   175
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                     14       0  100.00%
R/shift_ard_columns.R             77      13  83.12%   61, 69, 144-147, 150-156
R/shuffle_ard.R                  102       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      11  71.79%   88-98
R/utils.R                         24       0  100.00%
TOTAL                           1826     105  94.25%

Diff against main

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

Results for commit: 69a24ff

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

github-actions bot commented Aug 1, 2024

Unit Test Performance Difference

Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
ard_hierarchical 💔 $7.93$ $+1.52$ ard_hierarchical_works_without_by_variables

Results for commit a52a369

♻️ This comment has been updated with latest results.

@pawelru pawelru requested a review from ddsjoberg August 15, 2024 06:44
@cicdguy cicdguy enabled auto-merge (squash) August 22, 2024 11:32
@cicdguy cicdguy merged commit 50fd557 into main Aug 22, 2024
27 of 28 checks passed
@cicdguy cicdguy deleted the propagations-august-2024 branch August 22, 2024 11:36
@github-actions github-actions bot locked and limited conversation to collaborators Aug 22, 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