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

extend check and docs github workflows with a trigger on filter_panel_refactor branch #318

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

m7pr
Copy link
Contributor

@m7pr m7pr commented Jun 2, 2023

One of actions we aligned on during this discussion #313 was to setup automated testing for branches that for a longer time substituted main branch. In this case it's filter_panel_refactor branch.

@m7pr m7pr requested a review from a team June 2, 2023 20:20
@m7pr m7pr added the core label Jun 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2023

Unit Tests Summary

    1 files    29 suites   21s ⏱️
394 tests 394 ✔️ 0 💤 0
666 runs  666 ✔️ 0 💤 0

Results for commit c6405fa.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 2, 2023

badge

Code Coverage Summary

Filename                      Stmts    Miss  Cover    Missing
--------------------------  -------  ------  -------  -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
R/calls_combine_by.R              7       0  100.00%
R/choices_labeled.R              49      14  71.43%   19, 30, 35, 45-50, 62, 66-70
R/filter_state_api.R             17      17  0.00%    137-172
R/FilteredData-utils.R           98      43  56.12%   126-131, 216-263
R/FilteredData.R                505     267  47.13%   139, 249-253, 302-316, 368-382, 717-849, 877-879, 883-886, 890-894, 898-900, 929-935, 942-944, 946-947, 955, 961-966, 984-989, 1012-1020, 1023-1048, 1063-1078, 1130-1151, 1175
R/FilteredDataCDISC.R           157      28  82.17%   101, 108, 114, 118, 144-153, 257-264, 282-285, 291, 358-361
R/FilteredDataset.R             188      77  59.04%   103, 199, 298, 362-419, 461-463, 502-508, 545-555
R/FilteredDatasetDefault.R      106       8  92.45%   166-173
R/FilteredDatasetMAE.R          208      59  71.63%   28, 119, 251-327
R/FilterPanelAPI.R               23       8  65.22%   88, 119-125
R/FilterState-utils.R           155      14  90.97%   154, 283-289, 301-306
R/FilterState.R                 227      88  61.23%   114-115, 236, 362-408, 514-586
R/FilterStateChoices.R          169      96  43.20%   229-235, 248, 274-375
R/FilterStateDate.R             165     108  34.55%   201-207, 220, 224, 255-371
R/FilterStateDatettime.R        238     166  30.25%   221-227, 241, 245, 276-453
R/FilterStateEmpty.R             41      16  60.98%   132-158
R/FilterStateLogical.R          138      79  42.75%   212, 228-320
R/FilterStateRange.R            225     129  42.67%   316-322, 332-333, 360-517
R/FilterStates-utils.R           68       5  92.65%   96, 113, 160, 206, 227
R/FilterStates.R                218      93  57.34%   94-100, 161, 220, 310-504, 521-522, 531
R/FilterStatesDF.R              230      29  87.39%   196-198, 202-210, 212, 216-218, 220, 386-395, 454, 523
R/FilterStatesMAE.R             221     106  52.04%   30, 90-95, 234-356, 373
R/FilterStatesMatrix.R          217     108  50.23%   39, 71-76, 84-85, 87, 135-136, 222-340
R/FilterStatesSE.R              409     201  50.86%   24, 90-95, 128-133, 140-141, 143, 304-313, 348-355, 363-370, 398-567
R/include_css_js.R                5       5  0.00%    12-16
R/resolve_state.R                23       0  100.00%
R/utils.R                        39       2  94.87%   101-102
R/variable_types.R               48      33  31.25%   42-47, 57, 70-105
R/zzz.R                           6       6  0.00%    3-11
TOTAL                          4200    1805  57.02%

Diff against main

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

Results for commit: ee3eeed

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

Copy link
Contributor

@donyunardi donyunardi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@m7pr m7pr merged commit 9ba3569 into main Jun 5, 2023
@m7pr m7pr deleted the extend_gha_workflows@main branch June 5, 2023 07:40
@m7pr m7pr restored the extend_gha_workflows@main branch June 5, 2023 13:31
m7pr added a commit that referenced this pull request Jun 5, 2023
m7pr added a commit that referenced this pull request Jun 5, 2023
#318 had typos in the name of the desired branch : )
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