Skip to content

remove remotes field#1719

Merged
donyunardi merged 1 commit into
mainfrom
remove-remotes@main
Jun 29, 2026
Merged

remove remotes field#1719
donyunardi merged 1 commit into
mainfrom
remove-remotes@main

Conversation

@donyunardi

Copy link
Copy Markdown
Contributor

Removing remotes field since teal.slice 0.8.0 made it to CRAN.

@donyunardi
donyunardi requested a review from averissimo June 29, 2026 10:58
@donyunardi
donyunardi enabled auto-merge (squash) June 29, 2026 11:04
@github-actions

Copy link
Copy Markdown
Contributor

badge

Code Coverage Summary

Filename                          Stmts    Miss  Cover    Missing
------------------------------  -------  ------  -------  -----------------------------------------------------------------------------------------------------
R/after.R                            59      21  64.41%   42-52, 64, 69, 77-79, 81-89, 100, 104-105
R/checkmate.R                        24       0  100.00%
R/decorators-helpers.R               59       1  98.31%   97
R/dummy_functions.R                  61       2  96.72%   54, 56
R/include_css_js.R                   11       0  100.00%
R/init.R                            136       0  100.00%
R/module_bookmark_manager.R          99      54  45.45%   78-133
R/module_data_summary.R             177       8  95.48%   40, 50, 205, 236-240
R/module_filter_data.R               64       0  100.00%
R/module_filter_manager.R           210       7  96.67%   119-120, 316, 343, 355, 362-363
R/module_nested_tabs.R              411      42  89.78%   120, 178-183, 211, 315-330, 350-354, 409, 527-530, 534-537, 541-544
R/module_session_info.R              18       0  100.00%
R/module_snapshot_manager.R         276       9  96.74%   315-319, 386, 389-391
R/module_source_code.R               69       0  100.00%
R/module_teal_lockfile.R            131      53  59.54%   45-57, 60-62, 76, 86-88, 100-102, 110-119, 122, 124, 126-127, 142-146, 161-162, 177-186
R/module_teal_reporter.R            122       9  92.62%   60, 77-78, 81, 98, 128, 142, 144, 158
R/module_teal.R                     304      26  91.45%   130, 179, 235, 265-285, 315-316
R/module_transform_data.R           164      25  84.76%   77-81, 85-89, 106, 112, 119-120, 134-138, 188, 273-277
R/module_validate_error.R            73       0  100.00%
R/modules.R                         345      50  85.51%   170-174, 229-232, 332, 339, 353, 472, 570-576, 706-712, 725-733, 748-763, 796, 808-816
R/reporter_previewer_module.R        41      12  70.73%   41, 45, 68-85
R/teal_data_module-eval_code.R       23       0  100.00%
R/teal_data_module-within.R           7       0  100.00%
R/teal_data_module.R                 23       0  100.00%
R/teal_data_utils.R                  49       0  100.00%
R/teal_modifiers.R                   57       0  100.00%
R/teal_slices-store.R                29       0  100.00%
R/teal_slices.R                      48       2  95.83%   153-154
R/teal_transform_module.R            45       0  100.00%
R/TealAppDriver.R                   363     255  29.75%   68-73, 125, 133-162, 193-202, 212, 227-275, 300-575, 603-628, 683, 685-688, 707-721, 738-739, 750-767
R/utils.R                           291      49  83.16%   38, 408-457, 545-554
R/validate_inputs.R                  57       0  100.00%
R/validations.R                      58       0  100.00%
R/zzz.R                              23       1  95.65%   25
TOTAL                              3927     626  84.06%

Diff against main

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

Results for commit: 59bd5e4

Minimum allowed coverage is 80%

♻️ This comment has been updated with latest results

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Unit Tests Summary

  1 files   36 suites   2m 42s ⏱️
485 tests 417 ✅ 68 💤 0 ❌
752 runs  684 ✅ 68 💤 0 ❌

Results for commit 59bd5e4.

♻️ This comment has been updated with latest results.

@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Unit Test Performance Difference

Test Suite $Status$ Time on main $±Time$ $±Tests$ $±Skipped$ $±Failures$ $±Errors$
module_teal 💔 $115.32$ $+11.53$ $0$ $0$ $0$ $0$
shinytest2-disable_report 👶 $+0.39$ $+4$ $+4$ $0$ $0$
shinytest2-disable_src 👶 $+0.20$ $+2$ $+2$ $0$ $0$
shinytest2-show-rcode 👶 $+0.29$ $+3$ $+3$ $0$ $0$
shinytest2-teal_modifiers 👶 $+0.52$ $+7$ $+7$ $0$ $0$
shinytest2-validate_input 💚 $26.56$ $-25.99$ $-4$ $+2$ $0$ $0$
Additional test case details
Test Suite $Status$ Time on main $±Time$ Test Case
module_teal 💔 $0.59$ $+1.03$ teal_reporter_Add_to_report_button_contains_error_reason_when_module_s_server_returns_an_error
shinytest2-decorators 💀 $0.04$ $-0.04$ unnamed
shinytest2-disable_report 👶 $+0.11$ Add_to_report_button_is_not_disabled_by_default.
shinytest2-disable_report 👶 $+0.09$ Report_button_is_active_on_a_nested_module_by_default
shinytest2-disable_report 👶 $+0.09$ Report_button_is_disabled_on_a_module_changed_by_disable_report_
shinytest2-disable_report 👶 $+0.09$ Report_button_is_disabled_on_nested_modules_changed_by_disable_report_
shinytest2-disable_src 👶 $+0.10$ Show_R_Code_button_is_disabled_on_a_module
shinytest2-disable_src 👶 $+0.11$ Show_R_Code_is_disabled_on_nested_modules_changed_with_disable_src
shinytest2-reporter 💀 $0.02$ $-0.02$ unnamed
shinytest2-show-rcode 👶 $+0.10$ e2e_Module_with_Show_R_Code_has_code
shinytest2-show-rcode 👶 $+0.09$ e2e_Module_with_Show_R_Code_has_modal_with_two_dismiss_and_two_copy_to_clipboard_buttons
shinytest2-show-rcode 👶 $+0.09$ e2e_Module_with_Show_R_Code_initializes_with_visible_button
shinytest2-teal_data_module 💀 $0.02$ $-0.02$ unnamed
shinytest2-teal_modifiers 👶 $+0.07$ e2e_add_landing_modal_displays_landing_modal_on_app_startup
shinytest2-teal_modifiers 👶 $+0.07$ e2e_add_landing_modal_modal_can_be_dismissed
shinytest2-teal_modifiers 👶 $+0.07$ e2e_combined_modifiers_displays_all_customizations_when_chained_together
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_footer_displays_custom_footer_in_the_app
shinytest2-teal_modifiers 👶 $+0.08$ e2e_modify_header_displays_custom_header_in_the_app
shinytest2-teal_modifiers 👶 $+0.07$ e2e_modify_title_sets_custom_title_in_the_page_title_head_title_displays_custom_favicon_in_the_app
shinytest2-teal_modifiers 👶 $+0.08$ e2e_modify_title_sets_custom_title_in_the_page_title_head_title_displays_custom_title_in_the_app
shinytest2-validate_input 💚 $13.54$ $-13.45$ validate_input_shinytest2_sequence_of_errors_with_parallel_validation
shinytest2-validate_input 💚 $12.98$ $-12.78$ validate_input_shinytest2_stale_messages_are_not_shown

Results for commit 937c755

♻️ This comment has been updated with latest results.

@donyunardi
donyunardi merged commit 538ec13 into main Jun 29, 2026
28 checks passed
@donyunardi
donyunardi deleted the remove-remotes@main branch June 29, 2026 11:07
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 29, 2026
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.

2 participants