Skip to content

feat: allow for an empty selection in the titles and footers decorator - #47

Merged
kpagacz merged 6 commits into
mainfrom
title-footer-empty-selection
Jul 9, 2026
Merged

feat: allow for an empty selection in the titles and footers decorator#47
kpagacz merged 6 commits into
mainfrom
title-footer-empty-selection

Conversation

@kpagacz

@kpagacz kpagacz commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                                     Stmts    Miss  Cover    Missing
-----------------------------------------  -------  ------  -------  ---------
R/BlockConditions.R                             21      21  0.00%    23-61
R/create_rel_risk_transformator.R               42      42  0.00%    41-87
R/forest_plot_decorator.R                      164     164  0.00%    97-373
R/ggplot_decorator.R                           245     245  0.00%    105-376
R/merge_levels_transformator.R                 146     146  0.00%    11-240
R/or_filtering_transformator_view_model.R       27      27  0.00%    25-61
R/or_filtering_transformator.R                 282     282  0.00%    69-397
R/patchwork_plot_decorator.R                    46      46  0.00%    18-68
R/r_access_utilities.R                          46      46  0.00%    32-152
R/ReportManager.R                              401     401  0.00%    41-743
R/title_footer_decorator.R                     142     142  0.00%    105-254
R/tm_report_manager.R                          719     719  0.00%    31-896
R/watermark_decorator.R                         61      61  0.00%    23-87
TOTAL                                         2342    2342  0.00%

Diff against main

Filename                      Stmts    Miss  Cover
--------------------------  -------  ------  --------
R/title_footer_decorator.R      +12     +12  +100.00%
TOTAL                           +12     +12  +100.00%

Results for commit: e23773d

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@pposiada
pposiada self-requested a review June 23, 2026 11:15
@pposiada

Copy link
Copy Markdown
Collaborator

@kpagacz Would it be possible here to do a unit test?

@kpagacz
kpagacz merged commit 95dbfb7 into main Jul 9, 2026
22 checks passed
@kpagacz
kpagacz deleted the title-footer-empty-selection branch July 9, 2026 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants